Re: [quicwg/base-drafts] Limit RCID state (#3547)

[ianswett <[e-mail]>]

@ianswett commented on this pull request.

I left some specific suggestions, PTAL.

> @@ -1069,6 +1069,18 @@ to cease using the connection IDs when requested can result in connection
 failures, as the issuing endpoint might be unable to continue using the
 connection IDs with the active connection.
 
+An endpoint MAY elect to only send or retransmit RETIRE_CONNECTION_ID frames

We typically don't talk about retransmissions, so this read oddly to me.

> @@ -1069,6 +1069,18 @@ to cease using the connection IDs when requested can result in connection
 failures, as the issuing endpoint might be unable to continue using the
 connection IDs with the active connection.
 
+An endpoint MAY elect to only send or retransmit RETIRE_CONNECTION_ID frames
+with sequence numbers greater than or equal to the highest Retire Prior To field
+received minus its advertised active_connection_id_limit. This bounds the
+necessary state at the endpoint. For example, if an endpoint receives a Retire
+Prior To field of 7 and has an advertised active_connection_id_limit of 4, it
+may cease sending any RETIRE_CONNECTION_ID frame with sequence number less than
+3.
+
+Peers are discouraged from issuing updates of the Retire Prior To field before

```suggestion
Peers SHOULD NOT issue updates of the Retire Prior To field before
```

> +To. Doing so may result in the peer being uncertain if it is to receive
+additional packets with an old Connection ID.

```suggestion
To. Doing so may result in the peer being unwilling to immediately retire all the connection IDs, if retiring them would cause it to exceed its limit on outstanding RETIRE_CONNECTION_ID frames, as described above.
```

> +An endpoint MAY elect to only send or retransmit RETIRE_CONNECTION_ID frames
+with sequence numbers greater than or equal to the highest Retire Prior To field
+received minus its advertised active_connection_id_limit. This bounds the
+necessary state at the endpoint. For example, if an endpoint receives a Retire
+Prior To field of 7 and has an advertised active_connection_id_limit of 4, it
+may cease sending any RETIRE_CONNECTION_ID frame with sequence number less than
+3.

```suggestion
An endpoint MAY elect to limit the number of outstanding RETIRE_CONNECTION_ID frames
to bound the necessary state.  In order to allow a peer to retire all previously issued
connection IDs, this limit SHOULD be at least the active_connection_id_limit.
```

-- 
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/3547#pullrequestreview-382811114