Skip to content

Commit

Permalink
Merge pull request #6 from drewmarshburn/dbmig-fix
Browse files Browse the repository at this point in the history
Fix db credential logging exposure
  • Loading branch information
onelapahead authored Jan 20, 2022
2 parents 6aaa1ed + 10a88fb commit 8584099
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions charts/firefly/scripts/ff-db-migrations.sh
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@
# Install deps
apk add postgresql-client curl jq

echo "Provided connection string: '${PSQL_URL}'"

# Extract the database name from the end of the PSQL URL, and check it's there
DB_PARAMS=`echo ${PSQL_URL} | sed 's!^.*/!!'`
DB_NAME=`echo ${DB_PARAMS} | sed 's!?.*!!'`
Expand All @@ -36,7 +34,6 @@ fi

# Check we can connect to the PSQL server using the default "postgres" database
PSQL_SERVER=`echo ${PSQL_URL} | sed "s!${DB_PARAMS}!!"`postgres
echo "PSQL server URL: '${PSQL_SERVER}'"
until psql -c "SELECT 1;" ${PSQL_SERVER}; do
echo "Waiting for PSQL server connection..."
sleep 1
Expand Down

0 comments on commit 8584099

Please sign in to comment.