Skip to content

Commit

Permalink
Be explicit about avoiding flow control issues
Browse files Browse the repository at this point in the history
  • Loading branch information
DavidSchinazi committed Jun 16, 2022
1 parent d7d06b1 commit 0de86db
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions draft-ietf-masque-h3-datagram.md
Expand Up @@ -370,8 +370,9 @@ violation of these requirements MUST treat the HTTP message as malformed.

When processing capsules, a receiver might be tempted to accumulate the full
length of the capsule value in the data stream before handling it. This approach
can consume flow control in underlying layers, which might lead to deadlocks if
the capsule data exhausts the flow control window.
SHOULD be avoided, because it can consume flow control in underlying layers, and
that might lead to deadlocks if the capsule data exhausts the flow control
window.


## Error Handling
Expand Down Expand Up @@ -484,8 +485,9 @@ into memory.

Since QUIC DATAGRAM frames are required to fit within a QUIC packet,
implementations that reencode DATAGRAM capsules into QUIC DATAGRAM frames might
be tempted to accumulate the entire capsule before reencoding it. This can cause
flow control problems; see {{capsule-protocol}}.
be tempted to accumulate the entire capsule in the stream before reencoding it.
This SHOULD be avoided, because it can cause flow control problems; see
{{capsule-protocol}}.

Note that it is possible for an HTTP extension to use HTTP Datagrams without
using the Capsule Protocol. For example, if an HTTP extension that uses HTTP
Expand Down

0 comments on commit 0de86db

Please sign in to comment.