Skip to content

Fix WSGI Violations#332

Open
TimPansino wants to merge 7 commits intopgjones:mainfrom
TimPansino:fix-wsgi-violations
Open

Fix WSGI Violations#332
TimPansino wants to merge 7 commits intopgjones:mainfrom
TimPansino:fix-wsgi-violations

Conversation

@TimPansino
Copy link

Overview

  • Fix WSGIWrapper class implementation to more closely follow the WSGI spec.
  • Add detailed regression tests that showcase the existing bugs in hypercorn, and confirm they function according to the WSGI spec after these changes.

Related

Closes #331

See that issue for a detailed breakdown of the issue and the reason behind these changes.

@TimPansino
Copy link
Author

@pgjones could you give this a review? Detailed justification is in #331

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.

WSGIWrapper violates WSGI protocol

1 participant