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

[Mike Bishop <[e-mail]>]

MikeBishop 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.

I think the confusing piece here is that the sender of the GOAWAY is the receiver of the in-flight request/push.  This means to talk about the receipt of the request, but makes it sound like it's talking about the receiver of the GOAWAY.

I'm not sure we need to eliminate any paragraphs, but clarifying the terms would probably help a lot.  This section could previously rely on "client" and "server" to clearly imply who's doing what with requests and GOAWAY frames; now it's generalized and easy to lose track of the involved parties.

> +
+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.
+
+Endpoints MUST NOT initiate new requests or pushes on the connection with an
+identifier greater than or equal to the smallest identifier received in a GOAWAY
+frame.  Clients MAY establish a new connection to send additional requests.
+
+Some requests or pushes might already be in transit. 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;
+requests MAY be retried by the client on a different connection.  The endpoint
+that initiated these requests or pushes MAY cancel them.  It is RECOMMENDED that
+the receiving endpoint explicitly reject such requests (see

```suggestion
such requests be explicitly rejected (see
```

> +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.
+
+Endpoints MUST NOT initiate new requests or pushes on the connection with an
+identifier greater than or equal to the smallest identifier received in a GOAWAY
+frame.  Clients MAY establish a new connection to send additional requests.
+
+Some requests or pushes might already be in transit. 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;
+requests MAY be retried by the client on a different connection.  The endpoint
+that initiated these requests or pushes MAY cancel them.  It is RECOMMENDED that
+the receiving endpoint explicitly reject such requests (see
+{{request-cancellation}}) or pushes (see {{frame-cancel-push}}) in order to

```suggestion
{{request-cancellation}} and {{frame-cancel-push}}) upon receipt in order to
```

-- 
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#pullrequestreview-342835509