Re: [quicwg/base-drafts] Rewrite text about Version Negotiation (#1039)
martinduke <notifications@github.com> Tue, 16 January 2018 06:18 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 35AF5126C2F for <quic-issues@ietfa.amsl.com>; Mon, 15 Jan 2018 22:18:38 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7
X-Spam-Level:
X-Spam-Status: No, score=-7 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, RCVD_IN_DNSWL_HI=-5, 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 gp5pX5p5ygc7 for <quic-issues@ietfa.amsl.com>; Mon, 15 Jan 2018 22:18:37 -0800 (PST)
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2-ext7.iad.github.net [192.30.252.198]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0B873124D68 for <quic-issues@ietf.org>; Mon, 15 Jan 2018 22:18:37 -0800 (PST)
Date: Mon, 15 Jan 2018 22:18:36 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1516083516; bh=Y8mmQ6oGSkCKohJkMiEVE/+oH0Tsj6YUomiaL+aGe5I=; h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=oWifYt+zURRwwXw1ViM40UA/a8NOa9J+vw08UtrY8JXbNfAOKgJ1yHwA6d4jQAkWK eoOxiVtrwB2W91uQhNNSIL368SEq1tyoOFQ+ZHtQqbTQgp+IBL2t8tmJwk3fgDVWMz 0IQXALsED0dp6O1sX4zeKj5JtoqNTZdsJe1Ooqq0=
From: martinduke <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab4766deb6b99f855e7049b557e73b7d4dd0c48e1892cf0000000116755b3c92a169ce1115d834@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1039/review/88993669@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1039@github.com>
References: <quicwg/base-drafts/pull/1039@github.com>
Subject: Re: [quicwg/base-drafts] Rewrite text about Version Negotiation (#1039)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5a5d993c608c8_103d3f8177702f281647618"; 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
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/DnyjEFB3ol1WEQ7k553ciYVcUDs>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.22
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: Tue, 16 Jan 2018 06:18:38 -0000
martinduke commented on this pull request. > -0-RTT packets might be received prior to a Client Initial packet at a server. -If the version of these packets is acceptable to the server, it MAY buffer these -packets in anticipation of receiving a reordered Client Initial packet. +with an existing connection, or - for servers - potentially create a new +connection. + +Hosts handle packets that can be associated with an existing connection +according to the current state of that connection. Both short form packets +without connection IDs and long-form packets for connections that have +incomplete cryptographic handshakes are associated with an existing +connection using the tuple of source and destination IP addresses and ports. +Other packets are associated with existing connections using connection ID +the connection ID in the header; this might include connection IDs that were +advertised using NEW_CONNECTION_ID ({{frame-new-connection-id}}). + +Clients SHOULD discard any packet that cannot be associated with an existing The key word here is "Clients". This finishes up instructions for clients, and the rest of the section is about servers. That said, a week after I wrote this it took me a minute to notice that. Any suggestions on how to make the word "clients" and its precise meaning more salient? -- 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/1039#discussion_r161667969
- [quicwg/base-drafts] Rewrite text about Version N… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… MikkelFJ
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… MikkelFJ
- Re: [quicwg/base-drafts] Rewrite text about Versi… ianswett
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… ianswett
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… MikkelFJ
- Re: [quicwg/base-drafts] Rewrite text about Versi… Mike Bishop
- Re: [quicwg/base-drafts] Rewrite text about Versi… MikkelFJ
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Nick Banks
- Re: [quicwg/base-drafts] Rewrite text about Versi… MikkelFJ
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… MikkelFJ
- Re: [quicwg/base-drafts] Rewrite text about Versi… MikkelFJ
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… Marten Seemann
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Mike Bishop
- Re: [quicwg/base-drafts] Rewrite text about Versi… Mike Bishop
- Re: [quicwg/base-drafts] Rewrite text about Versi… Mike Bishop
- Re: [quicwg/base-drafts] Rewrite text about Versi… Mike Bishop
- Re: [quicwg/base-drafts] Rewrite text about Versi… Kazuho Oku
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… martinduke
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson
- Re: [quicwg/base-drafts] Rewrite text about Versi… Martin Thomson