Re: [quicwg/base-drafts] Bundle new client connection ID and path challenge in migration probes (#1730)

Christian Huitema <notifications@github.com> Mon, 10 September 2018 19:59 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 C4A6C130E27 for <quic-issues@ietfa.amsl.com>; Mon, 10 Sep 2018 12:59:29 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.01
X-Spam-Level:
X-Spam-Status: No, score=-3.01 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] 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 0892HsIJln4e for <quic-issues@ietfa.amsl.com>; Mon, 10 Sep 2018 12:59:27 -0700 (PDT)
Received: from o10.sgmail.github.com (o10.sgmail.github.com [167.89.101.201]) (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 CE604130F67 for <quic-issues@ietf.org>; Mon, 10 Sep 2018 12:59:27 -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=K8MEbh5Q78nVHvZ96EguDP8psEs=; b=OVy00Xgqp017SgPk xGFphpDBAiu/dJsoXdJbvAY/mQAlCGiaWbwCyC+urn2xIidqZQeKikOMmEgCmAzw qLiEyIbikUfQcf6hDRC0yrU6E4rYeuOXksYVwoOsMYQ/rD0fu+PIYqEq+hvUBXR0 +E+s1pHE3S2iOXr3CFJqWO8NKg4=
Received: by filter0738p1las1.sendgrid.net with SMTP id filter0738p1las1-29390-5B96CD1F-2 2018-09-10 19:59:27.109434661 +0000 UTC m=+339550.631930201
Received: from github-lowworker-56a5eb2.cp1-iad.github.net (unknown [192.30.252.33]) by ismtpd0023p1iad2.sendgrid.net (SG) with ESMTP id DxAiXQ65Q76e69woljqsOw for <quic-issues@ietf.org>; Mon, 10 Sep 2018 19:59:27.002 +0000 (UTC)
Received: from github.com (localhost [127.0.0.1]) by github-lowworker-56a5eb2.cp1-iad.github.net (Postfix) with ESMTP id EF2BFC00A5 for <quic-issues@ietf.org>; Mon, 10 Sep 2018 12:59:26 -0700 (PDT)
Date: Mon, 10 Sep 2018 19:59:27 +0000
From: Christian Huitema <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab0faee9ccde31233707a01c48b886c34833e639df92cf0000000117ae8f1e92a169ce155cc367@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1730/review/153938177@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1730@github.com>
References: <quicwg/base-drafts/pull/1730@github.com>
Subject: Re: [quicwg/base-drafts] Bundle new client connection ID and path challenge in migration probes (#1730)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b96cd1ee91ec_3e23fae460d45c44141c"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: huitema
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: l64QuQ2uJCcEyUykJbxN122A6QRmEpucztpreh3Pak1QP/Wm4sSfYIqDU4pYqL9v4FbrvON6cU/jRY o2yMJybvMDFxb/DbL2lgRglkho1FHf/d67R4PYHLgXwEGkSBdu/0vvwW7z88UvAAgi1uyW1gntVXxE uvMZTBzYbQpZGSqRRMG9/A2wC/0cJRf/B6vxccydSMxxmKzUEpaqoi7iNOJMyz3XBoECMbELCpLUG7 Y=
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/5rf_YcIbZvtk-44IwitTN4Wh36k>
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, 10 Sep 2018 19:59:30 -0000

huitema commented on this pull request.



> @@ -2366,11 +2366,16 @@ genuine migrations.  Changing port number can cause a peer to reset its
 congestion state (see {{migration-cc}}), so the port SHOULD only be changed
 infrequently.
 
-Endpoints could also have their activity correlated if their peers keep using
-the same destination connection ID after migration. Nodes that initiate a
-migration need to provide their peers with new connection IDs. The goal is
+Endpoints that use connection IDs with length greater than zero
+could also have their activity correlated if their peers keep using
+the same destination connection ID after migration. If a node receives
+packets through a newly used connection ID, it SHOULD select an

@mikkelfj the whole point of this PR is to say that and ensure that it works. Do you want to specify what the server should do if the client's probe arrives before a new connection ID is available? By your logic, we want to say that the server should wait before responding to the probe.

Then, do you want to enforce that? For example, suppose that a client sent a probe with a new CID, and receives a response with an already-in-use CID. Should it just shrug and continue, or signal a protocol violation? 

-- 
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/1730#discussion_r216453075