-
-
Notifications
You must be signed in to change notification settings - Fork 328
-
@jmtaysom was having some issues with idom.run when using Sanic as well as with Sanic's SharedClientStateServer (previously discussed here).
Beta Was this translation helpful? Give feedback.
All reactions
Replies: 2 comments 3 replies
-
@jmtaysom what version of Sanic are you using?
Beta Was this translation helpful? Give feedback.
All reactions
-
Sanic 21.12.1
Beta Was this translation helpful? Give feedback.
All reactions
-
IDOM presently requires sanic>19,<19.12 - see here for more info. The other async server implementations should work fine without this limited range of versions.
Again, apologies for all these limitations. I'm presently working on a PR which will address most of the problems we've talked about:
- Rework
idom.run - Remove
SharedClientStateServer - Fix limited support for
sanic - etc.
Beta Was this translation helpful? Give feedback.
All reactions
-
Also, apologies for the sweeping changes here, but SharedClientStateServer is slated for removal since it doesn't work with multi-process web servers. Instead we would need to provide a more robust solution that would work across processes and hosts - probably using something like Redis.
Beta Was this translation helpful? Give feedback.
All reactions
-
That makes a lot of sense. If it is slated for removal I would remove it from the beginning of the tutorial just to not confuse people.
Beta Was this translation helpful? Give feedback.
All reactions
-
👍 1