Re: [quicwg/base-drafts] Clients can send GOAWAY too (#3129)

[Kazuho Oku <[e-mail]>]

kazuho commented on this pull request.



> -stream is reset individually, or the connection terminates.  Servers MAY reject
-individual requests on streams below the indicated ID if these requests were not
-processed.
+connection and initiate a graceful connection close.  Endpoints initiate the
+graceful shutdown of a connection by sending a GOAWAY frame ({{frame-goaway}}).
+The GOAWAY frame contains an identifier that indicates to the receiver the range
+of requests or pushes that were or might be processed in this connection.  The
+server sends a client-initiated bidirectional Stream ID; the client sends a Push
+ID.  Requests or pushes with the indicated identifier or greater are rejected by
+the sender of the GOAWAY.  This identifier MAY be zero if no requests or pushes
+were processed.
+
+The information in the GOAWAY enables a client and server to agree on which
+requests or pushes were accepted prior to the connection shutdown.  Endpoints
+SHOULD abruptly terminate any requests or pushes that have identifiers greater
+than or equal to the smallest identifier sent in a GOAWAY frame.

> This is talking about the sending endpoint, though.

That's true, and I now notice that the sentence starting from line 708 is the one that's incorrect.

The paragraph starting from line 703 talks about what to do with messages that were already in transit. It is a generalization of an exiting paragraph, that states that the sender of GOAWAY SHOULD reset, the receiver of GOAWAY MAY. I assume that we want to retain those behaviors.

The second sentence of that paragraph that talks about the receiver is correct (i.e. "If the endpoint has already sent requests or pushes with an identifier greater than or equal to that received in a GOAWAY frame, those requests or pushes will not be processed"). Next sentence that starts from 708 should then talk about the sender, but it incorrectly states: _It is RECOMMENDED that the **receiving** endpoint explicitly reject such requests._

I think it is beneficial to retain a paragraph that concentrates on handling of the HTTP messages in transit, and therefore still think that removing this paragraph (starting from line 694) is the correct thing to do, _and also fixing the "receiving" in the sentence starting from line 708 to "sending."

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/pull/3129#discussion_r366512233