-
Notifications
You must be signed in to change notification settings - Fork 36
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
error on fresh install with simplejson #16
Comments
Ok, looks like this is some kind of known issue that requires a monkeypatch in the user libraries, since requests library is not solving the problem... psf/requests#4842 |
The real issue is with the requests library, so the way I fixed it is I got rid of the simplejson import within that library. this is not a good solution, actually, it's a terrible one. But it works and it solves the problem at the source rather than requiring a lot of upstream changes. |
@vshesh Oh... i uninstall SimpleJson.... im gonna look for what you say. Thanks for your work. |
I have the same problem, it will be fixed soon? changing the code of your copy of the library is terrible and not a practical solution |
My solution was to remove |
Description
installed berserk a few minutes ago
Describe what you were trying to get done.
Tell us what happened, what went wrong, and what you expected to happen.
error is happening somewhere deep in the simplejson code. uninstalling simplejson makes the function work, but I shouldn't need to do this - I use simplejson in many other places (and other libraries depend on it) and so this is a bug for me. I tried to debug it myself for a but but could not understand why this error was appearing - the parameters look fine.
The text was updated successfully, but these errors were encountered: