Re: [quicwg/base-drafts] Discard inconsistent packets (#2431)

Mike Bishop <> Thu, 07 February 2019 00:43 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id 9B91B130F97 for <>; Wed, 6 Feb 2019 16:43:02 -0800 (PST)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -11.149
X-Spam-Status: No, score=-11.149 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-4.553, 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 xpEDTK1WpfCH for <>; Wed, 6 Feb 2019 16:43:00 -0800 (PST)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 3118C130F96 for <>; Wed, 6 Feb 2019 16:42:56 -0800 (PST)
Date: Wed, 06 Feb 2019 16:42:54 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=pf2014; t=1549500174; bh=Sr3e4+4Ju84Bah7/WgZtrnScMstr0rKlSuzv8outaLs=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=ZrYAYAfV8SlFPSYw9Ecnc3z+Qw+HTibDw1YTlO8lzI1L5s/x6nFqvEiykPiXQjBJZ RGW5eC1jIslDDdZMJ/CB6roVRtpJV1vEUSYQsoRonitDmhMz4rKLkFESi30vg5iFj9 UNyqYT5HL65rasxdLq9xPoIibcELKVGeP5LUV7MM=
From: Mike Bishop <>
Reply-To: quicwg/base-drafts <>
To: quicwg/base-drafts <>
Cc: Subscribed <>
Message-ID: <quicwg/base-drafts/pull/2431/review/>
In-Reply-To: <quicwg/base-drafts/pull/>
References: <quicwg/base-drafts/pull/>
Subject: Re: [quicwg/base-drafts] Discard inconsistent packets (#2431)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c5b7f0eee875_33983f97044d45c415123e"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: MikeBishop
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: Thu, 07 Feb 2019 00:43:03 -0000

MikeBishop commented on this pull request.

> @@ -1010,8 +1010,10 @@ Endpoints can send a Stateless Reset ({{stateless-reset}}) for any packets that
 cannot be attributed to an existing connection. A stateless reset allows a peer
 to more quickly identify when a connection becomes unusable.
-Packets that are matched to an existing connection, but for which the endpoint
-cannot remove packet protection, are discarded.
+Packets that are matched to an existing connection are discarded if the packets
+are inconsistent with the state of that connection -- for example, if they
+indicate a different protocol version than that of the connection, or if the
+endpoint cannot remove packet protection.

Perhaps "if removal of packet protection fails" would exclude the case where it can't be attempted yet?

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