Description
Description
A customer ran into a HTTP/2 error scenario and asked why the real failure reason wasn't in the error message: grpc/grpc-dotnet#2361 (comment). Seeing this error is important for debugging because the server can trigger a GOAWAY and knowing the error code is important for understanding why the server closed the connection.
Scenario:
- Response data from a HTTP/2 request is being read from the response stream
- The underlying HTTP/2 connection is killed because of a GOAWAY form the server
- The user sees an unhelpful/deceptive message:
System.Net.Http.HttpIOException: The response ended prematurely while waiting for the next frame from the server. (ResponseEnded)
at System.Net.Http.Http2Connection.ThrowRequestAborted(Exception innerException)
at System.Net.Http.Http2Connection.Http2Stream.CheckResponseBodyState()
at System.Net.Http.Http2Connection.Http2Stream.TryReadFromBuffer(Span`1 buffer, Boolean partOfSyncRead)
at System.Net.Http.Http2Connection.Http2Stream.ReadDataAsync(Memory`1 buffer, HttpResponseMessage responseMessage, CancellationToken cancellationToken)
The error gives the impression that the request ended because the server unexpectedly stopped sending response data. The real reason there are no more frames is because the server closed the connection with a GOAWAY. That information (GOAWAY + enhance your calm protocol code) should be bubbled up to this error message.
Something like:
The response ended because the server ended the HTTP/2 connection. Error code: ENHANCE_YOUR_CALM.
Today the only way to figure out why the HTTP/2 request failed is to attach an event source listener. That displays:
ProcessIncomingFramesAsync,Frame 15: StreamId=0; Type=GoAway; Flags=None; PayloadLength=22.
ReadGoAwayFrame,lastStreamId=1, errorCode=EnhanceYourCalm
Setting up an event source listener is advanced and shouldn't be required to find out important error information like this.
Reproduction Steps
- Client starts an HTTP/2 request and awaits response headers
- Server returns response headers
- Client reads from response stream
- Server sends a GOAWAY
- Client gets the unhelpful error
Here is an example in logs:
Expected behavior
A more useful error message. It should say the response ended because the connection ended. It should include the GOAWAY error code.
Something like:
The response ended because the server ended the HTTP/2 connection. Error code: ENHANCE_YOUR_CALM.
Actual behavior
Unhelpful error message.
Regression?
No response
Known Workarounds
No response
Configuration
No response
Other information
No response