Re: [quicwg/base-drafts] Rework Retry packet (#1498)
Kazuho Oku <notifications@github.com> Fri, 29 June 2018 01:56 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 97A53130E2F for <quic-issues@ietfa.amsl.com>; Thu, 28 Jun 2018 18:56:54 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.01
X-Spam-Level:
X-Spam-Status: No, score=-8.01 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, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] 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 UOfs7ZMeLHBj for <quic-issues@ietfa.amsl.com>; Thu, 28 Jun 2018 18:56:53 -0700 (PDT)
Received: from out-4.smtp.github.com (out-4.smtp.github.com [192.30.252.195]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C9CD4120049 for <quic-issues@ietf.org>; Thu, 28 Jun 2018 18:56:52 -0700 (PDT)
Date: Thu, 28 Jun 2018 18:56:51 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1530237411; bh=mcgFbnvMmrMrqMGJS6+TlsdjBTUlhSXh0u0VIdzxAzw=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=RWkDk4EMN/L90eH6+E1cBiXnAOvkPCNQRDYMJqDXVVSXX/PcgsTOMkyEgjuUl+jHg blBTfyjlr7mYb1buwp/U5dIJrlNbGlzcdgjZXgonEkYVTqKJYsiVn3PjTVO6UDWBmn HzZDsKe/dqlB9jczk8581jJh7k8rHFthcYHaL7oo=
From: Kazuho Oku <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abbf6f0cca26fa665be7582c3940871fa39ae0082492cf00000001174d53e392a169ce14138c09@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1498/c401224459@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1498@github.com>
References: <quicwg/base-drafts/pull/1498@github.com>
Subject: Re: [quicwg/base-drafts] Rework Retry packet (#1498)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b3591e37edbd_36522af29f29af6085079"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: kazuho
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/cEetF_S7AVs34sHo3YrPNeqgqrs>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.26
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, 29 Jun 2018 01:56:56 -0000
> Open Question: would it make sense to normalize the encoding of Retry and Initial? The former includes a token without a length (because it's the last piece of the packet), whereas the latter includes a length-prefixed token. It might be easier to invert the ODCID and token on Retry. That would make the token processing more consistent. My preference goes to keeping the current format as-is, because IMO it is better to have fields that identify the connection at the top. Having ODCID above token allows implementations to bail out before dealing with the token, if ODCID does not match the expected value. > I'm not sure if we want to consider adding a token to 0-RTT and Handshake packets, but that would fully normalize things. I think that we should limit the existence of the token field to Initial packets. Having the field in Handshake packet does not make sense at all, because it is sent by the client after the path is validated using the token. Note also that we are trying hard to cram the certificate chain into the Handshake packets that can be sent in the first flight. Having an unnecessary field (even if it is empty) goes against that. Having the field in 0-RTT packet _might_ make some sense, but I oppose, because it allows two ways of implementing buffering of potentially reordered 0-RTT packets on the server side. One way (the only way which is possible now) is to buffer such packets without validating the paths. The other way that will be opened up by having the field is to buffer such packets after validating the paths. This flexibility will incentivize the clients to include token for every 0-RTT packets even though it reduces the space that can be used to carry application data. And that in turn disincentivises the server to buffer potentially reordered 0-RTT packets. To summarize, having a token field in 0-RTT packet will force us to end up in an suboptimal implementation, even though 0-RTT is about optimization. I do not think that we want to see such an outcome. -- 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/1498#issuecomment-401224459
- Re: [quicwg/base-drafts] Rework Retry packet (#14… janaiyengar
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Christian Huitema
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Kazuho Oku
- Re: [quicwg/base-drafts] Rework Retry packet (#14… ekr
- [quicwg/base-drafts] Rework Retry packet (#1498) Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… ekr
- Re: [quicwg/base-drafts] Rework Retry packet (#14… ianswett
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Mike Bishop
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… ianswett
- Re: [quicwg/base-drafts] Rework Retry packet (#14… ianswett
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Christian Huitema
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Mike Bishop
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Mike Bishop
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Marten Seemann
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Christian Huitema
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Marten Seemann
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… MikkelFJ
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Nick Banks
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson
- Re: [quicwg/base-drafts] Rework Retry packet (#14… Martin Thomson