You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
SETTINGS_NUM_PLACEHOLDERS (0x3): This value SHOULD be non-zero. The
default value is 16.
SETTINGS_MAX_HEADER_LIST_SIZE (0x6): The default value is unlimited.
It seems inconsistent to reuse 0x3 while forbidding other setting IDs from RFC 7540. SETTINGS_NUM_PLACEHOLDERS is not the same as SETTINGS_MAX_CONCURRENT_STREAMS.
The text was updated successfully, but these errors were encountered:
Hand-wavy similarity. It's a limit on the number of a given resource the peer is allowed to consume. I frankly don't care one way or the other -- if having the reuse causes more issues for anyone's implementation than changing the codepoint does, I'm happy to move it.
We explicitly forbid other HTTP settings because (presumably?) we want to catch bugs. By reusing 0x3, we allow for a potential bug when one side uses HTTP2 semantics -- likely due to some code reuse -- instead of HQ.
There are two consistent choices: either use another value instead of 0x3, or renumber HQ settings and not care about forbidding HTTP2 setting values.
Not so much bug-catching as to simplify life for people with shared code-bases. But sure, I don't have a problem with moving it. That would be a breaking change, though, so I'll save it for post-16.
From the spec:
It seems inconsistent to reuse 0x3 while forbidding other setting IDs from RFC 7540.
SETTINGS_NUM_PLACEHOLDERS
is not the same asSETTINGS_MAX_CONCURRENT_STREAMS
.The text was updated successfully, but these errors were encountered: