Skip to content

Worker threads WSGI Middleware Configurable#2052

Open
EyalMK wants to merge 2 commits intospec-first:mainfrom
EyalMK:configurable-flask-threads
Open

Worker threads WSGI Middleware Configurable#2052
EyalMK wants to merge 2 commits intospec-first:mainfrom
EyalMK:configurable-flask-threads

Conversation

@EyalMK
Copy link
Copy Markdown

@EyalMK EyalMK commented May 7, 2025

Fixes #1979.
"Make FlaskApp number of threads configurable"

Changes proposed in this pull request:

  • FlaskApp has an additional optional worker_threads constructor argument
  • worker_threads default to 10; identical to the default no. threads for WSGIMiddleware from a2wsgi

@EyalMK
Copy link
Copy Markdown
Author

EyalMK commented May 7, 2025

@RobbeSneyders Would much appreciate a review for this. Current state makes Connexion 3 and non-thread safe libraries totally incompatible...

@FelixSchwarz
Copy link
Copy Markdown
Contributor

@Ruwann This PR looks pretty simple as well. Any chance it could be included in 2.15.0?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make FlaskApp number of threads configurable

2 participants