Skip to content

[26.0] Fix WSGI middlewares returning strings#22108

Merged
mvdbeek merged 2 commits intogalaxyproject:release_26.0from
mvdbeek:fix_middleware_strings
Mar 14, 2026
Merged

[26.0] Fix WSGI middlewares returning strings#22108
mvdbeek merged 2 commits intogalaxyproject:release_26.0from
mvdbeek:fix_middleware_strings

Conversation

@mvdbeek
Copy link
Copy Markdown
Member

@mvdbeek mvdbeek commented Mar 13, 2026

We should remove the unused ones in dev.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

mvdbeek added 2 commits March 13, 2026 16:27
The error inner function was used as a WSGI sub-application but returned
a bare string. WSGI requires response bodies to be iterables of byte
strings. With a2wsgi converting to ASGI, h11 rejects the str, causing a
TypeError in starlette's BaseHTTPMiddleware.
@mvdbeek mvdbeek merged commit 3a56ddf into galaxyproject:release_26.0 Mar 14, 2026
53 of 57 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in Galaxy Dev - weeklies Mar 14, 2026
@ahmedhamidawan ahmedhamidawan modified the milestones: 26.1, 26.0 Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants