Re: [quicwg/base-drafts] Add note that middleboxes might change datagram boundaries (#3337)

Eric Kinnear <notifications@github.com> Fri, 17 January 2020 06:26 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 624441201AA for <quic-issues@ietfa.amsl.com>; Thu, 16 Jan 2020 22:26:14 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.596
X-Spam-Level:
X-Spam-Status: No, score=-6.596 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, HTML_IMAGE_ONLY_28=1.404, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, 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 19PPBEsTiE7z for <quic-issues@ietfa.amsl.com>; Thu, 16 Jan 2020 22:26:09 -0800 (PST)
Received: from out-24.smtp.github.com (out-24.smtp.github.com [192.30.252.207]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E463E120119 for <quic-issues@ietf.org>; Thu, 16 Jan 2020 22:26:08 -0800 (PST)
Received: from github-lowworker-39b4a70.va3-iad.github.net (github-lowworker-39b4a70.va3-iad.github.net [10.48.16.66]) by smtp.github.com (Postfix) with ESMTP id 0E8686A05A6 for <quic-issues@ietf.org>; Thu, 16 Jan 2020 22:26:08 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1579242368; bh=1YAWjPwfMf4N2LG8PKsTDwtnlmgQJdXZ/ilRYtHGG7Y=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=AnTarXTgRQH666MsvYSKkO79P3txiCom6UGSETlKwY/ZrwIY0U+F41ZI3ZRmwoZmZ ax8sqcgakOBmckiGo7SGRTv7Orbx5KEAmbd8wTP46B/KFRRMyB7u4cNEcstf/MzFCn iPlXFuYWIZxYaMnoVEG8aot6Sh0y/TceADerCM5E=
Date: Thu, 16 Jan 2020 22:26:07 -0800
From: Eric Kinnear <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK5QKXLZPI6DVAQ65Y54F2C77EVBNHHCBO77TE@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3337/review/344383110@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3337@github.com>
References: <quicwg/base-drafts/pull/3337@github.com>
Subject: Re: [quicwg/base-drafts] Add note that middleboxes might change datagram boundaries (#3337)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5e21537ff3af8_4c143fd2baacd95c692aa"; 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/MhpjkokeMUunkPz6Ytf-EcJThWM>
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: Fri, 17 Jan 2020 06:26:15 -0000

erickinnear commented on this pull request.



> @@ -3515,6 +3515,10 @@ A server MAY send a CONNECTION_CLOSE frame with error code PROTOCOL_VIOLATION in
 response to an Initial packet it receives from a client if the UDP datagram is
 smaller than 1200 bytes. It MUST NOT send any other frame type in response, or
 otherwise behave as if any part of the offending packet was processed as valid.
+Note that a middlebox might modify datagram boundaries when multiple QUIC
+packets have been coalesced into a single UDP datagram, causing the handshake to
+fail. This is considered a failure of the path to support datagrams of the size

I tend to agree with both the sentence, as well as your statements. :) 
Really what I'm looking for is "this is a failure on the part of the network and something we've decided to live with and if someone on the path does this, it's their fault" -- better wording welcome!

-- 
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/3337#discussion_r367785373