Unknown overflow error about sanic-session HOT 4 CLOSED

I now see that this i has to do with ujson, and that in rare instances, what is inside the session contains objects that can’t be reliably dumped. The doc implies that I can put anything in the session but it seems that this is not the case at all.

I am using ujson==1.35

from sanic-session.

PyPI say that current stable version for the ujson is 3.2.0. Can you upgrade to the latest version? Probably the problem is already fixed. Module doesn't use any specific ujson features. It was chosen as faster alternative for standard json library from Python. It was an important speedup when module was written for the first time. As far as I know standard json is better now, so probably we can just drop support for json and use it only if it already installed.

If it is more convenient way for you pull request will be appreciated.

Something like:

try:
    import ujson as json
except ModuleNotFoundError:
    import json

from sanic-session.

I can’t use ujson==3.2.0 right now because they have removed the ability to convert objects starting in v2+. This is technically doable if you would consider adding a custom serializer for the dump — but right now it will break my application if I simply upgrade ujson directly. (e.g. if dict contains datetime it will break immediately, whereas in 1.35 it will go through)

For example, there is no real reason why it must convert to json — if I am able to use pickle, for example. I have a full discussion on this matter on the sanic community here: https://community.sanicframework.org/t/mysterious-overflowerror-with-sanic-session/698/13?u=smlbiobot

And a quick solution would be for this library to support custom serializers (even if you only want to convert to json). Then I could get around this issue by doing custom json encoding to handle all of my use cases.

from sanic-session.

ujson is recommended dependency of Sanic framework itself. I think the more practical way for your case will be forking of sanic_session. Or take only parts you needed and implement for yourself. This library is very simple but doesn't fit to your setup. Probably you need only 10-20 lines of code to implement your own sessions.

I will close the issue. There is nothing to do with sanic_sessions. Ofc, if you want to implement pluggable encoder/decoder you can do it, but I don't think it is very necessary in this specific project.

from sanic-session.