Re: [quicwg/base-drafts] Allow not creating QPACK codec streams (#2529)
Dmitri Tikhonov <notifications@github.com> Wed, 20 March 2019 20:29 UTC
Return-Path: <noreply@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 AB423131226 for <quic-issues@ietfa.amsl.com>; Wed, 20 Mar 2019 13:29:37 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8
X-Spam-Level:
X-Spam-Status: No, score=-8 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_HI=-5, 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 BHXYftuWZhDY for <quic-issues@ietfa.amsl.com>; Wed, 20 Mar 2019 13:29:35 -0700 (PDT)
Received: from out-4.smtp.github.com (out-4.smtp.github.com [192.30.252.195]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1B0E4131162 for <quic-issues@ietf.org>; Wed, 20 Mar 2019 13:29:35 -0700 (PDT)
Date: Wed, 20 Mar 2019 13:29:34 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1553113774; bh=aOC+ClmoVYrtcoLnIh32yl91C0lhf3je8OvitZ/NmbM=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=Avwcw5x//9LNVad0SJaCEHxbauZxhntwJw3u9g5soEZiOwTu6VAqSAXbOxvW1IQ/h zdUot0NGnCkdQj4RQoFkUhe3MoqMTmh9wX5h6hBSANho1t/yGDQ8UiO6b8g16zPo2I /fbNm0FhSHKj3ScUmZZqeziI9QOq7t8tjC7iMT4I=
From: Dmitri Tikhonov <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abcb934b93b8459c951863b7f99b5a90b719bfae4f92cf0000000118aa64ae92a169ce1931c3e6@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/216964696@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_5c92a2ae1ae99_1a443f9a5e4d45bc2459f"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: dtikhonov
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/ZANg4szLXF_u1vvfNr7w7cX-9VY>
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: Wed, 20 Mar 2019 20:29:43 -0000
dtikhonov 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 > I think the current consensus regarding QPACK streams is that you cannot prevent their creation What does that mean -- "[one] cannot prevent their creation?" Surely, setting the limit low does just that. The general MO of this WG has been to assume that the implementers are capable of making sensible choices. Creating an unnecessary QPACK stream instead of the HTTP Control Stream when the allowance is just one stream is far from a subtle "gotcha." -- 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_r267532197
- [quicwg/base-drafts] Allow not creating QPACK cod… Alessandro Ghedini
- Re: [quicwg/base-drafts] Allow not creating QPACK… Dmitri Tikhonov
- Re: [quicwg/base-drafts] Allow not creating QPACK… Lucas Pardue
- Re: [quicwg/base-drafts] Allow not creating QPACK… ianswett
- Re: [quicwg/base-drafts] Allow not creating QPACK… Lucas Pardue
- Re: [quicwg/base-drafts] Allow not creating QPACK… Kazuho Oku
- Re: [quicwg/base-drafts] Allow not creating QPACK… Alessandro Ghedini
- Re: [quicwg/base-drafts] Allow not creating QPACK… Dmitri Tikhonov
- Re: [quicwg/base-drafts] Allow not creating QPACK… afrind
- Re: [quicwg/base-drafts] Allow not creating QPACK… Dmitri Tikhonov
- Re: [quicwg/base-drafts] Allow not creating QPACK… Lucas Pardue
- Re: [quicwg/base-drafts] Allow not creating QPACK… afrind
- Re: [quicwg/base-drafts] Allow not creating QPACK… Lucas Pardue
- Re: [quicwg/base-drafts] Allow not creating QPACK… afrind
- Re: [quicwg/base-drafts] Allow not creating QPACK… Lucas Pardue
- Re: [quicwg/base-drafts] Allow not creating QPACK… afrind
- Re: [quicwg/base-drafts] Allow not creating QPACK… Lucas Pardue
- Re: [quicwg/base-drafts] Allow not creating QPACK… Bence Béky
- Re: [quicwg/base-drafts] Allow not creating QPACK… Alessandro Ghedini
- Re: [quicwg/base-drafts] Allow not creating QPACK… Mike Bishop
- Re: [quicwg/base-drafts] Allow not creating QPACK… Bence Béky
- Re: [quicwg/base-drafts] Allow not creating QPACK… afrind
- Re: [quicwg/base-drafts] Allow not creating QPACK… Alessandro Ghedini
- Re: [quicwg/base-drafts] Allow not creating QPACK… Alessandro Ghedini
- Re: [quicwg/base-drafts] Allow not creating QPACK… Alessandro Ghedini
- Re: [quicwg/base-drafts] Allow not creating QPACK… Alessandro Ghedini
- Re: [quicwg/base-drafts] Allow not creating QPACK… afrind