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

[ekr <[e-mail]>]

@ekr commented on this pull request.

I think that first sentence needs *some* clarification, though perhaps you don't like mine.

> @@ -1069,6 +1069,16 @@ 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 SHOULD limit the state it commits to retiring connection IDs. An

I feel like "commits to" is unclear. 

Perhaps
"An endpoint should limit the number of connection IDs that it has retired locally and have not yet been acknowledged."

> @@ -1069,6 +1069,16 @@ 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 SHOULD limit the state it commits to retiring connection IDs. An
+endpoint SHOULD allow for sending and trackign a number of RETIRE_CONNECTION_ID

```suggestion
endpoint SHOULD allow for sending and tracking a number of RETIRE_CONNECTION_ID
```

-- 
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-409566191