Re: [quicwg/base-drafts] Disallow changes of table size after 0-RTT (#2276)
Mike Bishop <notifications@github.com> Mon, 07 January 2019 23:14 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 E87F212E7C1 for <quic-issues@ietfa.amsl.com>; Mon, 7 Jan 2019 15:14:35 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.064
X-Spam-Level:
X-Spam-Status: No, score=-8.064 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.065, 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 xYOaLsn0rJVc for <quic-issues@ietfa.amsl.com>; Mon, 7 Jan 2019 15:14:34 -0800 (PST)
Received: from out-3.smtp.github.com (out-3.smtp.github.com [192.30.252.194]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 137DB12D4ED for <quic-issues@ietf.org>; Mon, 7 Jan 2019 15:14:34 -0800 (PST)
Date: Mon, 07 Jan 2019 15:14:33 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1546902873; bh=5ocEaaIAsr6LXW52FA/g/zBVFurYYUqNUa1EmNA9lXE=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=O7PqaDaUZtNgDF6x8qxw5w+7S7jE76iSRTSOOm5m2VqaxpAH1MVHkc1RamXQyK0Eb 6X80+FXVPOFhwM7pNHTNPDujQuykMkwbt9Tf9Zi72EUh7julNHI1NWoUtVYYJ+iqQI hUukZkyJ1y6re384B/DHqPG4LbjJFEK4JxeFAApk=
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab612cba6591a5a94830f9a09f1ad8418d6f2e73d992cf00000001184b9f5992a169ce1789f586@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/2276/452116302@github.com>
In-Reply-To: <quicwg/base-drafts/issues/2276@github.com>
References: <quicwg/base-drafts/issues/2276@github.com>
Subject: Re: [quicwg/base-drafts] Disallow changes of table size after 0-RTT (#2276)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c33dd593d94e_29ef3f9bcf8d45b481972c"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: MikeBishop
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/AAOi69NgPyA03_YCfzDb_1oX9ew>
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: Mon, 07 Jan 2019 23:14:36 -0000
I don't feel strongly about the feature itself, but this would make QPACK differ from the "increase is permitted" pattern we've created. Assuming that SETTINGS arrives before any acknowledgements do and given that you can't evict unacknowledged entries from the dynamic table anyway, there's no way you could be wrapping under the old value of MaxEntries; increasing the max table size upon receipt of SETTINGS won't change the interpretation of anything you've sent. The problem scenario seems to be a SETTINGS frame that's delayed past some acknowledgements on the decoder stream -- a bar on evicting anything until SETTINGS is received seems like the simplest fix. -- 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/issues/2276#issuecomment-452116302
- [quicwg/base-drafts] Disallow changes of table si… Martin Thomson
- Re: [quicwg/base-drafts] Disallow changes of tabl… Mike Bishop
- Re: [quicwg/base-drafts] Disallow changes of tabl… afrind
- Re: [quicwg/base-drafts] Disallow changes of tabl… Martin Thomson
- Re: [quicwg/base-drafts] Disallow changes of tabl… Bence Béky
- Re: [quicwg/base-drafts] Disallow changes of tabl… Mike Bishop
- Re: [quicwg/base-drafts] Disallow changes of tabl… afrind