Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

database: database_connection.py: handle sensitive information, closes #71 #80

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

pcichowski
Copy link
Member

@pcichowski pcichowski commented Jan 19, 2023

Pull request review checklist πŸ“ βœ…

This is a checklist that helps conduct a pull request review properly. These points are required to merge the PR

  • There are no merge conflicts

  • The pull request is named accordingly (description of the functionality - it
    will be visible in the changelog later)

  • Every task requirement (from Jira) is done

  • The app runs, it boots up properly (e.g. no syntax errors)

  • The proposed functionality is working properly (you need to pull this branch
    and check for yourself)

  • Each function, class or functionality is properly documented
    (short description of what it does, parameters or return values if they need
    any explanation)

@pcichowski pcichowski requested a review from Mixss January 19, 2023 20:12
@pcichowski pcichowski merged commit 71da3c9 into staging Jan 19, 2023
@pcichowski pcichowski deleted the cleartext-db-password-fix branch January 19, 2023 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant