Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replace flask_sockets #19

Open
camoverride opened this issue Feb 16, 2021 · 0 comments
Open

Replace flask_sockets #19

camoverride opened this issue Feb 16, 2021 · 0 comments
Assignees
@camoverride
Labels
enhancement New feature or request question Further information is requested

Comments

@camoverride
Copy link
Owner

camoverride commented Feb 16, 2021 •
edited
Loading

Investigate whether flask_sockets should be replaced with the real deal: https://cloud.google.com/appengine/docs/flexible/python/using-websockets-and-session-affinity

Also, set websockets to timeout after 5 minutes - they shouldn't be open forever

Also, thread_lock is used in app.py but I'm not sure if it's necessary when app is dockerized. better understand thread locking
@camoverride camoverride added enhancement New feature or request question Further information is requested labels Feb 16, 2021
@camoverride camoverride self-assigned this Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@camoverride camoverride

Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@camoverride