Date: Tue, 19 Mar 2019 10:57:18 -0700
From: Kazuho Oku <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+0166e4ab28c51f3fbbfb8a6f2fca87a0cf9675c1b9b5fcee92cf0000000118a8ef7e92a169ce1931c3e6@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2529/review/216327132@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2529@github.com>
References: <quicwg/base-drafts/pull/2529@github.com>
Subject: Re: [quicwg/base-drafts] Allow not creating QPACK codec streams
 (#2529)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5c912d7ec34c2_2a8c3fec984d45bc524131";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/u6dUlSMQbcwowtYJVdVmog6rio4>


----==_mimepart_5c912d7ec34c2_2a8c3fec984d45bc524131
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

kazuho commented on this pull request.



>  HTTP/3 endpoints contain a QPACK encoder and decoder. Each endpoint MUST
-initiate a single encoder stream and decoder stream. Receipt of a second
-instance of either stream type be MUST treated as a connection error of type
+initiate at most one encoder stream and one decoder stream. Receipt of a second

Agrees to @dtikhonov. Maybe adding something like following immediately after could be sufficient. "Endpoints MAY avoid creating encoder streams if they are not going to be used. They MAY avoid or defer the creation of a decoder stream."

-- 
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/2529#discussion_r267024119
----==_mimepart_5c912d7ec34c2_2a8c3fec984d45bc524131
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

<p><b>@kazuho</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/quicwg/base-drafts/pull/2529#discussion_r267024119">draft-ietf-quic-qpack.md</a>:</p>
<pre style='color:#555'>&gt;  HTTP/3 endpoints contain a QPACK encoder and decoder. Each endpoint MUST
-initiate a single encoder stream and decoder stream. Receipt of a second
-instance of either stream type be MUST treated as a connection error of type
+initiate at most one encoder stream and one decoder stream. Receipt of a second
</pre>
<p>Agrees to <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=4528576" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/dtikhonov">@dtikhonov</a>. Maybe adding something like following immediately after could be sufficient. "Endpoints MAY avoid creating encoder streams if they are not going to be used. They MAY avoid or defer the creation of a decoder stream."</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/2529#discussion_r267024119">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AWbkq4gQ2sweHhvt4WOspnq9UbuSacUAks5vYST-gaJpZM4b78tQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/AWbkq0W1YDXQHS-LkWmc6LhuW8QBeNOWks5vYST-gaJpZM4b78tQ.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/quicwg/base-drafts","title":"quicwg/base-drafts","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/quicwg/base-drafts"}},"updates":{"snippets":[{"icon":"PERSON","message":"@kazuho commented on #2529"}],"action":{"name":"View Pull Request","url":"https://github.com/quicwg/base-drafts/pull/2529#discussion_r267024119"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/pull/2529#discussion_r267024119",
"url": "https://github.com/quicwg/base-drafts/pull/2529#discussion_r267024119",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>
----==_mimepart_5c912d7ec34c2_2a8c3fec984d45bc524131--