Re: [quicwg/base-drafts] Strengthen 2119 language around tokens. (#2124)

MikkelFJ <> Wed, 12 December 2018 14:51 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id ADFDA129533 for <>; Wed, 12 Dec 2018 06:51:26 -0800 (PST)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -8.056
X-Spam-Status: No, score=-8.056 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-1.46, 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: (amavisd-new); dkim=pass (1024-bit key)
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id Wc27_qR9AfMs for <>; Wed, 12 Dec 2018 06:51:25 -0800 (PST)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 11564127B92 for <>; Wed, 12 Dec 2018 06:51:25 -0800 (PST)
Date: Wed, 12 Dec 2018 06:51:24 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=pf2014; t=1544626284; bh=7HdohZPvFtleBzyCQd1OKpv5l8UCDqLcvC6H9meS+jA=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=PxFMHw0yWKuRYWsWzFa10Gn9TC/4L+taw9vQIN+jOEtUH881zvVi3xvTuAbLMl/t8 opf/p3C+Zm6e+mIMw/NU7G/oUoxox/eCjs0goargvbnCQQ0fSTDFxPgwvC2v+Z9q9p CS7/P1PipG39S/L/iah6lR+dnld9calvNAQa7fAA=
From: MikkelFJ <>
Reply-To: quicwg/base-drafts <>
To: quicwg/base-drafts <>
Cc: Subscribed <>
Message-ID: <quicwg/base-drafts/pull/2124/review/>
In-Reply-To: <quicwg/base-drafts/pull/>
References: <quicwg/base-drafts/pull/>
Subject: Re: [quicwg/base-drafts] Strengthen 2119 language around tokens. (#2124)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c11206c135bd_37853f8d8fed45c094828"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: mikkelfj
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
Archived-At: <>
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Wed, 12 Dec 2018 14:51:27 -0000

mikkelfj commented on this pull request.

> @@ -1627,7 +1627,7 @@ interface.  A client needs to start the connection process over if it migrates
 prior to completing the handshake.
 When a server receives an Initial packet with an address validation token, it
-SHOULD attempt to validate it, unless it has already completed address
+MUST attempt to validate it, unless it has already completed address

MUST implies that it must fail if it cannot validate. During server cluster upgrades that might not be feasible. It could lead to consistent connection failures as long at the client has a token to try. It is in the servers interest to generally attempt validation so SHOULD seems reasonable to me.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: