Re: [quicwg/base-drafts] Allow not creating QPACK codec streams (#2529)

Lucas Pardue <notifications@github.com> Tue, 19 March 2019 12:51 UTC

Return-Path: <bounces+848413-a050-quic-issues=ietf.org@sgmail.github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 484CF12782C for <quic-issues@ietfa.amsl.com>; Tue, 19 Mar 2019 05:51:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3
X-Spam-Level:
X-Spam-Status: No, score=-3 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_IMAGE_ONLY_32=0.001, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id rDXt205QeBQ1 for <quic-issues@ietfa.amsl.com>; Tue, 19 Mar 2019 05:51:57 -0700 (PDT)
Received: from o1.sgmail.github.com (o1.sgmail.github.com [192.254.114.176]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D1AD0131263 for <quic-issues@ietf.org>; Tue, 19 Mar 2019 05:51:56 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com; h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe; s=s20150108; bh=JVYxdhejRNJ11PbkWcOBqT2ARG0=; b=f6KCL8pA27YGxtw7 BIPKK5UzuWZCszSPNcszwMXjGzsKnyCqzpcVRyb9PCWYXhtlB+Hc1BWxGDp7IlTx arOP8Q88n3Qk0CHHPSScitwOBh5XdKkpyfBKIzFMfnS17UI5j5W3v0UfRn3djHIA pDbPdbyOiqJWxDTBXXiFmJUswj8=
Received: by filter0431p1iad2.sendgrid.net with SMTP id filter0431p1iad2-1139-5C90E5EB-26 2019-03-19 12:51:55.42741817 +0000 UTC m=+327931.400047255
Received: from github-lowworker-dcc078e.cp1-iad.github.net (unknown [192.30.252.44]) by ismtpd0009p1iad2.sendgrid.net (SG) with ESMTP id 6DxF9Uj1QY-FEFgMu7deLg for <quic-issues@ietf.org>; Tue, 19 Mar 2019 12:51:55.360 +0000 (UTC)
Received: from github.com (localhost [127.0.0.1]) by github-lowworker-dcc078e.cp1-iad.github.net (Postfix) with ESMTP id 62AEA2C008D for <quic-issues@ietf.org>; Tue, 19 Mar 2019 05:51:55 -0700 (PDT)
Date: Tue, 19 Mar 2019 12:51:55 +0000
From: Lucas Pardue <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab9308453f40623f7cb57d4bf673a1e1d33dda7f1392cf0000000118a8a7eb92a169ce1931c3e6@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/216132851@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_5c90e5eb60d62_2c8d3fc75c0d45b81111fe"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: LPardue
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
X-SG-EID: l64QuQ2uJCcEyUykJbxN122A6QRmEpucztpreh3Pak3aRze9979jZht+CIo0wwa2OMQOTYK3XHO25r w4FCKles3FoHCh69nKkwzTwL4htDCTkUF5WHLWQQ9d1S7BDolP5zgZpxwHThQFS6P2tmXuMZiTRaZQ muk6AQzE8W4RHehqu6unJqJpW9xRDl5iYhw3NHgiIb21znTS0OYtJa9D97IWnU126vNU52mj3vIInI M=
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/XvZNXsIBqoQkzdwL8jMSwSKsH6A>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 19 Mar 2019 12:52:00 -0000

LPardue commented on this pull request.



>  HTTP_WRONG_STREAM_COUNT. These streams MUST NOT be closed. Closure of either
 unidirectional stream type MUST be treated as a connection error of type
 HTTP_CLOSED_CRITICAL_STREAM.
 
+An endpoint MAY avoid creating its own encoder stream if the maximum size of
+the dynamic table permitted by the peer is zero.
+
+An endpoint MAY avoid creating its own decoder stream if the maximum size of
+its own dynamic table is zero.
+
+An endpoint MUST allow its peer to create both encoder and decoder streams

Going back over some of the discussion thread on this topic, was the preference to do lazy loading of streams in response to SETTINGS? In which case, when an endpoint receives a SETTINGS_QPACK_MAX_TABLE_CAPACITY it should send a QUIC MAX_STREAMS to allow the peer to open the encoder stream, and hope that the peer has also given enough streams to open the decoder stream.

Its hard to tell from the different threads what the consensus was.

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