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.

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

Make test requests execute WSGI middleware as expected #988

Closed
wants to merge 1 commit into from

Conversation

@jstasiak
Copy link
Contributor

@jstasiak jstasiak commented Aug 16, 2016

The comment in the test explains what I believe to be an unexpected
behavior that this patch attemps to correct. I'm not sure if this is the
best place to place the iterator-exhausting code, I'll be happy to move
it around.

The comment in the test explains what I believe to be an unexpected
behavior that this patch attemps to correct. I'm not sure if this is the
best place to place the iterator-exhausting code, I'll be happy to move
it around.
@davidism
Copy link
Member

@davidism davidism commented Nov 25, 2018

You can pass buffered=True to open (get, post, etc.) and it will ensure each response body is read. But I think you're right that buffered=False only makes sense for the final response.

@davidism davidism added this to the 0.15 milestone Nov 25, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants