Re: [quicwg/base-drafts] Added more test to NCID section (#3535)

martinduke <> Fri, 20 March 2020 00:35 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id C5B8E3A1359 for <>; Thu, 19 Mar 2020 17:35:30 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -1.554
X-Spam-Status: No, score=-1.554 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_IMAGE_ONLY_20=1.546, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, SPF_HELO_NONE=0.001, 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 jG9z_X6JbiVe for <>; Thu, 19 Mar 2020 17:35:23 -0700 (PDT)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 86BD63A1354 for <>; Thu, 19 Mar 2020 17:35:07 -0700 (PDT)
Date: Thu, 19 Mar 2020 17:34:57 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=pf2014; t=1584664497; bh=uXHe1WhJz8xIZpExdXo6zemy41UoLhZid/0ZPS7xBTg=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=sICT9vPChAi1PbT1n70hTdiTPlQCY7+soPMtpEAHRf9L78RAHdYul+xaHXTj8UwJO Ap/QZ3XgSC2+f2FcDhcfeBnvezPb7Kbh4R9KzQYuyTNre8rJIV/d/2FhefaS+XuHTW WUGOG9GdYnABoMnG/WGDjIrFDEn1R8OhX7zaI3sU=
From: martinduke <>
Reply-To: quicwg/base-drafts <>
To: quicwg/base-drafts <>
Cc: Subscribed <>
Message-ID: <quicwg/base-drafts/pull/3535/review/>
In-Reply-To: <quicwg/base-drafts/pull/>
References: <quicwg/base-drafts/pull/>
Subject: Re: [quicwg/base-drafts] Added more test to NCID section (#3535)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5e740fb1d4efd_62043fbd45ccd96c50059"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: martinduke
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: Fri, 20 Mar 2020 00:35:31 -0000

martinduke commented on this pull request.

> @@ -5571,6 +5571,12 @@ Retire Prior To field MUST be less than or equal to the Sequence Number field.
 Receiving a value greater than the Sequence Number MUST be treated as a
 connection error of type FRAME_ENCODING_ERROR.
+Endpoints that receive a NEW_CONNECTION_ID frame SHOULD NOT retire connection

I am happy to jettison this paragraph to get the other bit through. The point is to say "here is a stupid thing to do and don't do it". If there's some other wording, or some other place to put it that's better, I'm fine with that too.

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