-
Notifications
You must be signed in to change notification settings - Fork 84
Migrate Notifications server to python #198 #526
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
base: master
Are you sure you want to change the base?
Conversation
7712185
to
92aae1b
Compare
Unfortunately, I was able to reproduce socketify.py issue #141, which causes the server to segfault due to improper memory management within the library. As a result, I can no longer use socketify.py for the notifications server. The only viable alternative is the Python websockets library, though it is slower. I will rewritte the server to use the websocket liblary, however I'm uncertain if the performance will be satisfactory. |
Migrated the server. Used socketify.py for high performance WebSockets. The server configuration variables and deployment remain the same. Additionally the server is now tested using the normal Django test suite.