Opened 7 years ago

Closed 6 years ago

#477 closed editorial (incorporated)

Pipelining language

Reported by: mnot@… Owned by: draft-ietf-httpbis-p1-messaging@…
Priority: normal Milestone: 23
Component: p1-messaging Severity: In WG Last Call
Keywords: Cc:

Description

Section 6.6 of p1 states:

A server that sends a close connection option MUST initiate a lingering close of the connection after it sends the response containing close. The server MUST NOT process any further requests received on that connection.

A client that receives a close connection option MUST cease sending requests on that connection and close the connection after reading the response message containing the close; if additional pipelined requests had been sent on the connection, the client SHOULD assume that they will not be processed by the server.

The last sentence can be interpreted one of two ways: 1) The client SHOULD assume the additional pipelined requests will NOT be processed by the server and therefore can happily re-try them knowing the server has not processed the previous ones.

2) The client SHOULD NOT assume the additional pipelined requests will be processed (which implies the client simply can not know whether the server has processed them or not).

As the client has no way of knowing whether the server may have processed them or not (e.g. the client may be talking to a proxy that has already relayed the pipelined requests to the origin and done so before the proxy was aware that it wanted to close the connection on this response) I would suggest rewording the last sentence quoted above:

OLD:

the client SHOULD assume that they will not be processed by the server.

NEW:

the client SHOULD NOT assume that they will be processed by the server.

Change History (1)

comment:1 Changed 6 years ago by fielding@…

  • Milestone changed from unassigned to 23
  • Resolution set to incorporated
  • Status changed from new to closed

From [2258]:

(editorial) #477 : tis better to SHOULD NOT assume than to assume not.

Note: See TracTickets for help on using tickets.