Date: Fri, 24 Jul 2020 11:41:02 -0700
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+AFTOJK4NOCNZYYJ34QUSJAN5E4GD5EVBNHHCPHGA74@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3955/review/455107098@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3955@github.com>
References: <quicwg/base-drafts/pull/3955@github.com>
Subject: Re: [quicwg/base-drafts] Clarify HTTP/2 setting parameter reservation
 (#3955)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5f1b2b3e37c0f_7eeb3fb905ccd9604244e8";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/DPF_6XGks3QUv3LD34YXqYbgawk>


----==_mimepart_5f1b2b3e37c0f_7eeb3fb905ccd9604244e8
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

@MikeBishop commented on this pull request.



> @@ -2316,20 +2320,26 @@ SETTINGS_HEADER_TABLE_SIZE:
 
 SETTINGS_ENABLE_PUSH:
 : This is removed in favor of the MAX_PUSH_ID which provides a more granular
-  control over server push.
+  control over server push.  Specifying a setting with the identifier 0x2

Nothing in this appendix is normative, by design.  The normative requirement is in the body of the document, and this appendix simply notes that there is a prohibition.  I could add a reference to the prohibition, if that would help.

-- 
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/3955#discussion_r460225544
----==_mimepart_5f1b2b3e37c0f_7eeb3fb905ccd9604244e8
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

<p></p>
<p><b>@MikeBishop</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/quicwg/base-drafts/pull/3955#discussion_r460225544">draft-ietf-quic-http.md</a>:</p>
<pre style='color:#555'>&gt; @@ -2316,20 +2320,26 @@ SETTINGS_HEADER_TABLE_SIZE:
 
 SETTINGS_ENABLE_PUSH:
 : This is removed in favor of the MAX_PUSH_ID which provides a more granular
-  control over server push.
+  control over server push.  Specifying a setting with the identifier 0x2
</pre>
<p>Nothing in this appendix is normative, by design.  The normative requirement is in the body of the document, and this appendix simply notes that there is a prohibition.  I could add a reference to the prohibition, if that would help.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/quicwg/base-drafts/pull/3955#discussion_r460225544">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AFTOJK5ZPUWW7SMLW7B6GTDR5HIT5ANCNFSM4PF3NEMQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AFTOJK3SN3KAYU4PN3OT5FLR5HIT5A5CNFSM4PF3NEM2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGODMQGEGQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/pull/3955#discussion_r460225544",
"url": "https://github.com/quicwg/base-drafts/pull/3955#discussion_r460225544",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>
----==_mimepart_5f1b2b3e37c0f_7eeb3fb905ccd9604244e8--