- Notifications
You must be signed in to change notification settings - Fork50
Added client encoding to adapted SQL parameters.#159
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This is an awesome start. Would it be possible to run a test that raises the encoding error so we can verify the reverse circumstance here? |
…cute_sql. It now raises ValueError if client_encoding does not match the database encoding.
It is hard to catch the error as a specific encoding error. I ended up adding a keyword argument |
I ran some more tests and got errors for adapted parameters that do not have the |
Fix & test for#158.