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 5B1CA120842
 for <quic-issues@ietfa.amsl.com>; Wed, 15 Jan 2020 08:26:28 -0800 (PST)
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 SAEFaNuRdvJf for <quic-issues@ietfa.amsl.com>;
 Wed, 15 Jan 2020 08:26:21 -0800 (PST)
Received: from out-5.smtp.github.com (out-5.smtp.github.com [192.30.252.196])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by ietfa.amsl.com (Postfix) with ESMTPS id BD8C912004A
 for <quic-issues@ietf.org>; Wed, 15 Jan 2020 08:26:21 -0800 (PST)
Date: Wed, 15 Jan 2020 08:26:21 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
 s=pf2014; t=1579105581;
 bh=tXJIxQVX4f17Srw5UMsTqYEyXKn7RU5R2BxdhPsgmqE=;
 h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
 List-Archive:List-Post:List-Unsubscribe:From;
 b=VAu25R5m+H1NxUM0jRedHaGKSDX2eypG9lfGuNLliy0DOHqkfsLFWbgH4GfdKig+w
 IBhwm2OLJfgYtQy7YzR7clgehacLi+pLe0s+sk/y3WclT0li9nK22zmsR7uNvH5a0z
 ONJh6X/So8RVR4+7WD15UPesrnxaQDZ3eK2mj2SQ=
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+AFTOJK7PJWGPLTIK4L6VD5V4FRX23EVBNHHCBPAYFQ@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3338/review/343349112@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3338@github.com>
References: <quicwg/base-drafts/pull/3338@github.com>
Subject: Re: [quicwg/base-drafts] GOAWAY means no more (#3338)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5e1f3d2db167_61a13fb3822cd95c1727c8";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: MikeBishop
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/6haYSLK1hJrDzj0vYDiMLxg4CK0>
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: Wed, 15 Jan 2020 16:26:28 -0000


----==_mimepart_5e1f3d2db167_61a13fb3822cd95c1727c8
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

MikeBishop commented on this pull request.



> +Before closing the connection, a GOAWAY frame MAY be sent to allow the client to
+retry some requests.  Including the GOAWAY frame in the same packet as the QUIC
 CONNECTION_CLOSE frame improves the chances of the frame being received by
 clients.

Admittedly, it's best effort.  This also implies that there might be value in permitting the application layer to drain whatever data has arrived, even if a CONNECTION_CLOSE has also been received.

-- 
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/3338#discussion_r366974294
----==_mimepart_5e1f3d2db167_61a13fb3822cd95c1727c8
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

<p><b>@MikeBishop</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/quicwg/base-drafts/pull/3338#discussion_r366974294">draft-ietf-quic-http.md</a>:</p>
<pre style='color:#555'>&gt; +Before closing the connection, a GOAWAY frame MAY be sent to allow the client to
+retry some requests.  Including the GOAWAY frame in the same packet as the QUIC
 CONNECTION_CLOSE frame improves the chances of the frame being received by
 clients.
</pre>
<p>Admittedly, it's best effort.  This also implies that there might be value in permitting the application layer to drain whatever data has arrived, even if a CONNECTION_CLOSE has also been received.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/quicwg/base-drafts/pull/3338?email_source=notifications&amp;email_token=AFTOJK6GSQ6UEIISBLGRUZ3Q542K3A5CNFSM4KGJHRJKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCR3RO6A#discussion_r366974294">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AFTOJK5CWFBR6C4NIBFPUPLQ542K3ANCNFSM4KGJHRJA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AFTOJK5XSIR4NVO4VKV7NSTQ542K3A5CNFSM4KGJHRJKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCR3RO6A.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/pull/3338?email_source=notifications\u0026email_token=AFTOJK6GSQ6UEIISBLGRUZ3Q542K3A5CNFSM4KGJHRJKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCR3RO6A#discussion_r366974294",
"url": "https://github.com/quicwg/base-drafts/pull/3338?email_source=notifications\u0026email_token=AFTOJK6GSQ6UEIISBLGRUZ3Q542K3A5CNFSM4KGJHRJKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCR3RO6A#discussion_r366974294",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>
----==_mimepart_5e1f3d2db167_61a13fb3822cd95c1727c8--

