Skip to content

Conversation

@ronag
Copy link
Member

@ronag ronag commented Jul 13, 2019

Stop writing to request object after response. Instead just dump data. See, #27916 (comment) and #28668.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@ronag
Copy link
Member Author

ronag commented Jul 13, 2019

ping @lpinca @mcollina

@ronag ronag changed the title streams: stop piping if dest has resposne streams: pipeline stop piping if dest has response Jul 13, 2019
@ronag
Copy link
Member Author

ronag commented Jul 13, 2019

Not sure what the corresponding would be for http2

@ronag ronag force-pushed the fix-stop-writing-after-response branch from 182b99c to 560bf39 Compare July 13, 2019 20:50
@ronag
Copy link
Member Author

ronag commented Jul 14, 2019

Not relevant anymore, see #28668 (comment).

@mcollina
Copy link
Member

Closing because of #28672 (comment)

@mcollina mcollina closed this Jul 14, 2019
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.

2 participants