Re: [quicwg/base-drafts] Clarify text around preferred address (#3589)
Eric Kinnear <notifications@github.com> Mon, 11 May 2020 17:36 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 5F7C83A0BA9 for <quic-issues@ietfa.amsl.com>; Mon, 11 May 2020 10:36:14 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.101
X-Spam-Level:
X-Spam-Status: No, score=-3.101 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, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, 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 Kcrr2jQgslw0 for <quic-issues@ietfa.amsl.com>; Mon, 11 May 2020 10:35:56 -0700 (PDT)
Received: from out-5.smtp.github.com (out-5.smtp.github.com [192.30.252.196]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C2B063A0B68 for <quic-issues@ietf.org>; Mon, 11 May 2020 10:35:55 -0700 (PDT)
Received: from github-lowworker-1b8c660.ash1-iad.github.net (github-lowworker-1b8c660.ash1-iad.github.net [10.56.18.59]) by smtp.github.com (Postfix) with ESMTP id EB3DE961787 for <quic-issues@ietf.org>; Mon, 11 May 2020 10:35:53 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1589218553; bh=JycTkzBhMPMaZGWb4V1Shyth8PPxxuLP1ptI7o8pKIk=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=VaH1g3hY84mrSiGPzUpmp4WvCZSl+Mdqi6CiOMtVEvqYzdxCzU4qv8sgEsIIYiMkn 6OMiIw0iEP8npYvKcp0idcbubads67pUxYO5bAEJ0qrTT/pBgoIloSPkXk81gE732G 7/YoYqKWAZAt+uk44IzEhb0oOyqKHe25XTgTj98k=
Date: Mon, 11 May 2020 10:35:53 -0700
From: Eric Kinnear <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJKYDW4YPRWKH57U3T554YVW7TEVBNHHCH4LVNY@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3589/review/409384786@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3589@github.com>
References: <quicwg/base-drafts/pull/3589@github.com>
Subject: Re: [quicwg/base-drafts] Clarify text around preferred address (#3589)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5eb98cf9de04f_1c253fc9740cd960350a"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: erickinnear
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/jhBY5xQjbuE6N-dUhGBU5zps2K0>
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, 11 May 2020 17:36:17 -0000
@erickinnear commented on this pull request. > @@ -1084,10 +1084,10 @@ Sending a RETIRE_CONNECTION_ID frame indicates that the connection ID will not be used again and requests that the peer replace it with a new connection ID using a NEW_CONNECTION_ID frame. -As discussed in {{migration-linkability}}, endpoints limit the use of a -connection ID to a single network path where possible. Endpoints SHOULD retire -connection IDs when no longer actively using the network path on which the -connection ID was used. +As discussed in {{migration-linkability}}, endpoints MUST limit the use of a +connection ID to packets sent from a single local address. Endpoints SHOULD +retire connection IDs when they are no longer actively using the network path on +which the connection ID was used. > Perhaps the problem is the use of "network path". I would agree, I think we've been trying to steer away from "network path" in favor of things from the perspective of each endpoint. > The point about NAT rebinding/inadvertent migration here is not relevant and somewhat distracting. The NAT rebinding is something that's been covered elsewhere already, I believe, but every review that we've had of this text has resulted in someone raising their hand to say "don't forget about NAT rebinding here", so have kept text in to settle that. > The issue is that with NAT your perspective and others may different. When we distilled this down, we went from the perspective of the requirements that are placed on each endpoint to help cope with this. > This text is now contradictory, because I can send from A -> B and A->C That shouldn't be the case: ``` An endpoint MUST NOT reuse a connection ID when sending from more than one local address, ... ``` ``` Similarly, an endpoint MUST NOT reuse a connection ID when sending to more than one destination address. ``` I think we can change the text above to not mention a "network path" and instead talk about the destination address, since that's what we really mean in this case. -- 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/3589#discussion_r423205675
- [quicwg/base-drafts] Clarify text around preferre… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson
- Re: [quicwg/base-drafts] Clarify text around pref… Kazuho Oku
- Re: [quicwg/base-drafts] Clarify text around pref… Kazuho Oku
- Re: [quicwg/base-drafts] Clarify text around pref… Mike Bishop
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson
- Re: [quicwg/base-drafts] Clarify text around pref… ianswett
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… David Schinazi
- Re: [quicwg/base-drafts] Clarify text around pref… ekr
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson
- Re: [quicwg/base-drafts] Clarify text around pref… ekr
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson
- Re: [quicwg/base-drafts] Clarify text around pref… ianswett
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… ianswett
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson
- Re: [quicwg/base-drafts] Clarify text around pref… Jana Iyengar
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… ekr
- Re: [quicwg/base-drafts] Clarify text around pref… ekr
- Re: [quicwg/base-drafts] Clarify text around pref… Eric Kinnear
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson
- Re: [quicwg/base-drafts] Clarify text around pref… Martin Thomson