Fix: moved ssl_mode parameter for mysql backend engine #5771
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed change
The introduced change corrects a bug in which the
ssl_mode
parameter was not taken into account by the MySQLdb engine. Thessl_mode
parameter is not taken from thessl
parameter as can be verified in MySQLdb code and documentation. The easiest way to confirm the error is to set thePAPERLESS_DBSSLMODE
environment variable toREQUIRED
and start the MySQL database by setting therequire_secure_transport=on
variable. Althoughssl_mode
was set toREQUIRED
the MySQLdb engine does not see this setting and we get an errorMySQLdb.OperationalError: (3159, 'Connections using insecure transport are prohibited while --require_secure_transport=ON.')
If the
ssl_mode
parameter is set correctly, this error should not occur.Type of change
Checklist:
pre-commit
hooks, see documentation.