Scott.Idem/OSIT-Mailman3
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: Do not log full database url.

Browse Source 
Since the database contain credentials, do not log them after
changing from `mysql://` to `mysql+pymysql://`.

Fixes #642
This commit is contained in:
Abhilash Raj
2023-08-28 20:52:51 +05:30
parent 13411cbe62
commit 528ddba4d2
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
core/docker-entrypoint.sh
View File

@@ -67,7 +67,7 @@ function setup_database () {
# Translate mysql:// urls to mysql+mysql:// backend: # Translate mysql:// urls to mysql+mysql:// backend:
if [[ "$DATABASE_URL" == mysql://* ]]; then if [[ "$DATABASE_URL" == mysql://* ]]; then
DATABASE_URL="mysql+pymysql://${DATABASE_URL:8}" DATABASE_URL="mysql+pymysql://${DATABASE_URL:8}"
echo "Database URL was automatically rewritten to: $DATABASE_URL" echo "Database URL prefix was automatically rewritten to: mysql+pymysql://"
fi fi
# If DATABASE_CLASS is not set, guess it for common databases: # If DATABASE_CLASS is not set, guess it for common databases: