Fix verify_in_ssl log statement which no longer has access to SSL params #202
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.
Ticket
#200
Changes
Removes log statement that relies on variables no longer in psycopg 3.
Context for reviewers
The upgrade to psycopg3 got this error during migrations:
AttributeError: 'ConnectionInfo' object has no attribute 'ssl_attribute_names'
- https://github.com/navapbc/platform-test-flask/actions/runs/6275581568/job/17043454952These aren't available anymore and looking through the connection info object, I don't see any equivalent. Looking at the docs for version 2, these attributes say
Only available if psycopg was built with libpq >= 9.5
https://www.psycopg.org/docs/extensions.html#psycopg2.extensions.ConnectionInfo.ssl_attributeWhile the docs for psycopg3 mention libpq as being the backing library it uses, I don't see any mention of these parameters in their docs. https://www.psycopg.org/psycopg3/docs/api/connections.html#psycopg.Connection.pgconn - so seems to just be gone entirely?
Testing
Updated tests