Skip to content

[Experimental] PoC of websockets using flask-sock #1358

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
Closed
victorgarcia98 wants to merge 12 commits into from
Closed

[Experimental] PoC of websockets using flask-sock #1358

victorgarcia98 wants to merge 12 commits into from

Conversation

@victorgarcia98
Copy link
Contributor

@victorgarcia98 victorgarcia98 commented Mar 6, 2025 •
edited by VladIftime
Loading

Description

POC of using websockets using the Flask-Sock library.

TODO:

  • Test multiple connections
  • How can we handle authentication/authorization?
  • What's the optimal way to implement tests?

This PR closes #1378

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on code under GPL or other license that is incompatible with FlexMeasures

poc of websockets using flask-sock
87fef87 
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
@victorgarcia98 victorgarcia98 self-assigned this Mar 6, 2025
apply pre-commit
902c8cd 
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
@Flix6x Flix6x added the API label Mar 6, 2025
@Flix6x
Copy link
Contributor

Flix6x commented Mar 14, 2025

@Ahmad-Wahid Want to give this a test run on your WS test server?

poc: test if we can get the app context
b54c530 
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
@Flix6x Flix6x mentioned this pull request Mar 18, 2025
Closed
@Flix6x Flix6x linked an issue Mar 18, 2025 that may be closed by this pull request
Set up simple WS #1374
Open
This was referenced Mar 18, 2025
Closed
Closed
Victor Garcia Reolid and others added 6 commits March 25, 2025 18:51
@Flix6x Flix6x mentioned this pull request Jun 23, 2025
Open
Flix6x and others added 3 commits June 25, 2025 16:35
@Flix6x
Merge remote-tracking branch 'refs/remotes/origin/main' into exp/webs…
b3fe923 
…ockets/flask-sock-poc
@Flix6x
style: black
6fe7fa9 
Signed-off-by: F.N. Claessen <felix@seita.nl>
@VladIftime
chore: Update flake8 configuration to ignore C901 complexity warnings
ad46309 
Added C901 to the list of ignored warnings in setup.cfg to accommodate code complexity requirements.
@nhoening
Copy link
Contributor

nhoening commented Sep 5, 2025

Can this be closed?

@nhoening
Copy link
Contributor

nhoening commented Sep 22, 2025

Development continued in this branch.

@nhoening nhoening closed this Sep 22, 2025
@Flix6x
Copy link
Contributor

Flix6x commented Dec 8, 2025

Moved to #1852.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nhoening nhoening Awaiting requested review from nhoening

@Flix6x Flix6x Awaiting requested review from Flix6x

Assignees

@victorgarcia98 victorgarcia98

Labels

API

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Set up simple WS Stress test response times when clients become more chatty

5 participants

@victorgarcia98 @Flix6x @nhoening @VladIftime