Skip to content

fix: Do not log full database url.#646

Merged
maxking merged 2 commits intomainfrom
fix-642
Aug 28, 2023
Merged

fix: Do not log full database url.#646
maxking merged 2 commits intomainfrom
fix-642

Conversation

@maxking
Copy link
Copy Markdown
Owner

@maxking maxking commented Aug 28, 2023

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

Fixes #642

maxking and others added 2 commits August 28, 2023 20:52
Since the database contain credentials, do not log them after
changing from `mysql://` to `mysql+pymysql://`.

Fixes #642
@maxking maxking added the bug label Aug 28, 2023
@maxking maxking merged commit fe74e61 into main Aug 28, 2023
@maxking maxking deleted the fix-642 branch August 28, 2023 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Leaking Credentials

1 participant