Re: [quicwg/base-drafts] introduce a max_connection_ids transport parameter (#1998)
Marten Seemann <notifications@github.com> Fri, 21 December 2018 02:40 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 941B6130DCB for <quic-issues@ietfa.amsl.com>; Thu, 20 Dec 2018 18:40:23 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.661
X-Spam-Level:
X-Spam-Status: No, score=-6.661 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_28=1.404, 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 tadEQbXqdQgW for <quic-issues@ietfa.amsl.com>; Thu, 20 Dec 2018 18:40:21 -0800 (PST)
Received: from out-7.smtp.github.com (out-7.smtp.github.com [192.30.252.198]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A7FEA130DC4 for <quic-issues@ietf.org>; Thu, 20 Dec 2018 18:40:21 -0800 (PST)
Date: Thu, 20 Dec 2018 18:40:20 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1545360020; bh=GJ0aWvvXoxMystvt+E4gkb5mDZLir3uuh/OGP/3JW/0=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=eSkUUvbzaXyHu5KX5296pXOxs+B8AevnQlbj7PSClmvcEifKmK409c+lt+wXU7rJu bmWy+ikv9KdiTiYwFTW50YxYMcqyV4TfyTIbwx4cPKbW0q2GcrLeJIgOW5tQusbpaK QW3AVTgCPNgwP85EP96uTS7z7p/p1O+ctG+PpfQA=
From: Marten Seemann <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab6ecdd2ec0d6e5a25ada4236663c47cf374445f4492cf000000011834149492a169ce16a7e5d6@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1998/review/187256272@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1998@github.com>
References: <quicwg/base-drafts/pull/1998@github.com>
Subject: Re: [quicwg/base-drafts] introduce a max_connection_ids transport parameter (#1998)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c1c5294d739c_45133fa85ccd45bc140137"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: marten-seemann
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/8PKrbp9bk9XJ6WX3C1t9DcVH0oE>
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: Fri, 21 Dec 2018 02:40:24 -0000
marten-seemann commented on this pull request. > @@ -3981,6 +3979,12 @@ A client MUST NOT include an original connection ID, a stateless reset token, or a preferred address. A server MUST treat receipt of any of these transport parameters as a connection error of type TRANSPORT_PARAMETER_ERROR. +max_connection_ids (0x000e): + +: The maximum number of connection IDs that the peer is willing to store. + This value includes only connection IDs sent in NEW_CONNECTION_ID frames. I'm not sure why enforcing this creates pressure to increase the max. Sure, you need to advertise a value that actually makes sense (i.e. you should request as many CIDs as you think you'll need), and not rely on the peer overflowing this limit (which is exactly what we're trying to avoid in the first place). -- 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/1998#discussion_r243475549
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… MikkelFJ
- [quicwg/base-drafts] introduce a max_connection_i… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… MikkelFJ
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… erickinnear
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… MikkelFJ
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… erickinnear
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… janaiyengar
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… MikkelFJ
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… Christian Huitema
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… janaiyengar
- Re: [quicwg/base-drafts] introduce a max_connecti… Dmitri Tikhonov
- Re: [quicwg/base-drafts] introduce a max_connecti… Kazuho Oku
- Re: [quicwg/base-drafts] introduce a max_connecti… Kazuho Oku
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… janaiyengar
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Martin Thomson
- Re: [quicwg/base-drafts] introduce a max_connecti… erickinnear
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Christian Huitema
- Re: [quicwg/base-drafts] introduce a max_connecti… Christian Huitema
- Re: [quicwg/base-drafts] introduce a max_connecti… Christian Huitema
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Christian Huitema
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… Christian Huitema
- Re: [quicwg/base-drafts] introduce a max_connecti… Martin Thomson
- Re: [quicwg/base-drafts] introduce a max_connecti… MikkelFJ
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… Martin Thomson
- Re: [quicwg/base-drafts] introduce a max_connecti… Kazuho Oku
- Re: [quicwg/base-drafts] introduce a max_connecti… Kazuho Oku
- Re: [quicwg/base-drafts] introduce a max_connecti… Martin Thomson
- Re: [quicwg/base-drafts] introduce a max_connecti… martinduke
- Re: [quicwg/base-drafts] introduce a max_connecti… martinduke
- Re: [quicwg/base-drafts] introduce a max_connecti… Mike Bishop
- Re: [quicwg/base-drafts] introduce a max_connecti… Martin Thomson
- Re: [quicwg/base-drafts] introduce a max_connecti… Jana Iyengar
- Re: [quicwg/base-drafts] introduce a max_connecti… Kazuho Oku
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… Jana Iyengar
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Jana Iyengar
- Re: [quicwg/base-drafts] introduce a max_connecti… Martin Thomson
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… ianswett
- Re: [quicwg/base-drafts] introduce a max_connecti… Marten Seemann
- Re: [quicwg/base-drafts] introduce a max_connecti… Martin Thomson
- Re: [quicwg/base-drafts] introduce a max_connecti… Martin Thomson