IETF Logo Mail Archive

Re: [quicwg/base-drafts] Allow PING in Initial/Handshake? (#3034)

Kazuho Oku <notifications@github.com> Tue, 08 October 2019 23:22 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 80E7D1200B6 for <quic-issues@ietfa.amsl.com>; Tue, 8 Oct 2019 16:22:56 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.382
X-Spam-Level:
X-Spam-Status: No, score=-6.382 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_24=1.618, 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 n9bWWi33f3kk for <quic-issues@ietfa.amsl.com>; Tue, 8 Oct 2019 16:22:55 -0700 (PDT)
Received: from out-20.smtp.github.com (out-20.smtp.github.com [192.30.252.203]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 33D1B120046 for <quic-issues@ietf.org>; Tue, 8 Oct 2019 16:22:55 -0700 (PDT)
Date: Tue, 08 Oct 2019 16:22:54 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1570576974; bh=7didfz/gId+Tr2HoIDVx0926x2wCPA5Eoygk1jVcGBw=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=QEho2ShhSQ/PAs2EzAEmuI5XtX2LouHgSmzjGaDWBBxm2cqfva8gNfbFeVkvyG4ud dzbb7Bvtc2+C5WTvBhDn8ltsaL264yydVlKjulQWF9rDRbbRg+i8QJwer5Y7swpYwV +e0bvzwgpfbu0BXsXfr1ivAQgRhgry0QnmPuN2uA=
From: Kazuho Oku <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJKY4APORC6KJB2HJ3TN3VJGM5EVBNHHB23NCOM@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/3034/539745308@github.com>
In-Reply-To: <quicwg/base-drafts/issues/3034@github.com>
References: <quicwg/base-drafts/issues/3034@github.com>
Subject: Re: [quicwg/base-drafts] Allow PING in Initial/Handshake? (#3034)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5d9d1a4e88aaf_4fde3fd14c4cd96871125"; 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/Lpog7VDBkw7mro0fSSV8b2nu8f4>
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: Tue, 08 Oct 2019 23:22:56 -0000

@martinduke 
> * introduce special logic to spread out the data into two packets with a CRYPTO frame (gross)

IIUC, CRYPTO(off=0,len=0) can be used as a replacement for PING. There is no need to have special logic to spread out the data. So it's not as gross as it seems.

That said, considering the fact that we have PING in 1-RTT even though CRYPTO(off=0,len=0) can be used instead in 1-RTT also, I do not think there is a reason to argue *against* having PINGs for Initial and Handshake too.

-- 
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/issues/3034#issuecomment-539745308

v2.23.0 | Report a Bug | By Email