Re: [quicwg/base-drafts] Include handshake anti-deadlock logic in pseudocode (#2281)

Benjamin Saunders <> Sun, 31 March 2019 07:10 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id DF05C1201F2 for <>; Sun, 31 Mar 2019 00:10:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -6.597
X-Spam-Status: No, score=-6.597 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_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 HJbuI4ZaZf0x for <>; Sun, 31 Mar 2019 00:10:58 -0700 (PDT)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 0D85512017A for <>; Sun, 31 Mar 2019 00:10:58 -0700 (PDT)
Date: Sun, 31 Mar 2019 00:10:56 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=pf2014; t=1554016256; bh=hsOaMiojaCZqvo/+eOnRf2suQkD/dJfpNEKh0VnCFvM=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=RFoodn5+7qHscEtpVZ6ag+6MJTcvG/bOP9peNa5ug1jnRHoCIl6hkL6JGe7Ven3n/ tAvQ40egfqbhL085duXYVkZj9pf4NHlDPAf0w5EpAUqab6nFl601JsD2mhD6qDv/LV G3W8V1bAkTBE0SxoQw2nYtIee7h/R78oA2rrHsQk=
From: Benjamin Saunders <>
Reply-To: quicwg/base-drafts <>
To: quicwg/base-drafts <>
Cc: Subscribed <>
Message-ID: <quicwg/base-drafts/pull/2281/review/>
In-Reply-To: <quicwg/base-drafts/pull/>
References: <quicwg/base-drafts/pull/>
Subject: Re: [quicwg/base-drafts] Include handshake anti-deadlock logic in pseudocode (#2281)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5ca06800d6ca6_d8b3fbcb54d45c01875a"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: Ralith
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: Sun, 31 Mar 2019 07:11:00 -0000

Ralith commented on this pull request.

>      // Crypto retransmission timeout.
+  else if (endpoint is client without 1-RTT keys):
+    if (handshake keys available)

If the availability of Handshake keys isn't a strong enough condition for that, what should be used instead? I'd be inclined to just fall back on `SendOnePacket` and let the nontrivial padding logic be implied by the text elsewhere, but that's where we started for this fragment.

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