Skip to content

Clients should RST_STREAM(NO_ERROR) if not half-closed when receiving trailers #2163

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.

Sign up for GitHub

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

Jump to bottom
Closed
ejona86 opened this issue Aug 16, 2016 · 2 comments
Closed

Clients should RST_STREAM(NO_ERROR) if not half-closed when receiving trailers #2163

ejona86 opened this issue Aug 16, 2016 · 2 comments
Assignees
@carl-mastrangelo
Labels
bug cross-language-gotchas
Milestone
1.12

Comments

@ejona86
Copy link
Member

ejona86 commented Aug 16, 2016 •
edited
Loading

This is the client-side version of #2162, to handle the case when intermediaries are in play. Note that the RST_STREAM can be elided if the server sent a RST_STREAM after the EOS=true. But the client can't assume the server will send such a RST_STREAM.
@ejona86 ejona86 added the bug label Aug 16, 2016
@ejona86 ejona86 added this to the 1.1 milestone Aug 16, 2016
@ejona86 ejona86 mentioned this issue Aug 16, 2016
Open
@ejona86 ejona86 mentioned this issue Sep 8, 2016
Closed
@ejona86 ejona86 modified the milestones: Next, 1.1 Jan 17, 2017
@ejona86
Copy link
Member Author

ejona86 commented Aug 9, 2018

@ericgribkoff, was this resolved?

@ericgribkoff
Copy link
Contributor

Yes, OkHttp and Netty (after #4222) clients send a RST_STREAM in this scenario.

@ejona86 ejona86 modified the milestones: Next, 1.12 Aug 9, 2018
@ejona86 ejona86 mentioned this issue Sep 7, 2018
Open
@lock lock bot locked as resolved and limited conversation to collaborators Nov 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@carl-mastrangelo carl-mastrangelo

Labels
bug cross-language-gotchas
Projects
None yet
Milestone
1.12
Development

No branches or pull requests
3 participants
@ejona86 @ericgribkoff @carl-mastrangelo