-
-
Notifications
You must be signed in to change notification settings - Fork 340
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
UnicodeDecodeError in sql.py: leads to 500 internal error #85
Comments
as a temporary fix you can change model_factory.py:261 from |
@trik I appreciate the helpful temporary solution. I can apply it for now if that works for everyone. @ChristopherRabotin I will honestly have to look into official UTF-8 support. If anyone has time for a pull request I can be happy to take a look at it and merge it in. |
@avelis I was already thinking to a pull request but i don't understand well enough those lines.. the UnicodeDecodeError is never raised, the code inside the try/except does nothing but an assignment. we could keep my patch, not ignoring errors and if UDE is raised cause the sql has some binary parameter, use a different encoding to log (maybe b64) |
After a few days of testing when I opened this issue, we ended up not being able to use silk in production as it would lead to a substantial increase in the number of database queries making the main functionalities of the platform unavailable. |
It seems that silk does not support UTF-8 characters in SQL queries.
The text was updated successfully, but these errors were encountered: