Re: [quicwg/base-drafts] http deadlock with stream FIN choices (#1972)
Mike Bishop <notifications@github.com> Wed, 07 November 2018 02:43 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 20D6E127332 for <quic-issues@ietfa.amsl.com>; Tue, 6 Nov 2018 18:43:25 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.47
X-Spam-Level:
X-Spam-Status: No, score=-8.47 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.47, 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] 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 tOrAtIn3qICe for <quic-issues@ietfa.amsl.com>; Tue, 6 Nov 2018 18:43:23 -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 F110E126CB6 for <quic-issues@ietf.org>; Tue, 6 Nov 2018 18:43:22 -0800 (PST)
Date: Tue, 06 Nov 2018 18:43:22 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1541558602; bh=M+OtvDpQaEkPiVTgTWCDQmFPkDTLfxE/jBZPFM0YTsk=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=i3wx7bH9rk8kSOQMnWfolMFY7nlJo9GMjvBo2nRCYg/sJJHFHJcy4GdvPGNPiAIJJ SIcdzwltYUC7v+LUL5N1WuiKc6dK4KwLs6Xg7cZTaZ4oUoEtuiVWTgWz2OlEQXh/cz F+ZPH7qoweUFqmiQefIlJEmR+Fu35nLYF8p6nE18=
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abfbe4a15b1d58d8f713cc533727790bac0696b58892cf0000000117fa134a92a169ce1683c825@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1972/436484647@github.com>
In-Reply-To: <quicwg/base-drafts/issues/1972@github.com>
References: <quicwg/base-drafts/issues/1972@github.com>
Subject: Re: [quicwg/base-drafts] http deadlock with stream FIN choices (#1972)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5be2514a267a2_31503fa2c18d45c0116797"; 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/6dCI2SZqKI4YPl86xcFpQoUfCZg>
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, 07 Nov 2018 02:43:25 -0000
@martinduke filed this issue after discussing with me; while he encountered it with HTTP/0.9 in the interop, the issue applies to HQ as well. What the draft currently says is: > An HTTP request/response exchange fully consumes a bidirectional QUIC stream. After sending a request, a client closes the stream for sending; after sending a final response, the server closes the stream for sending and the QUIC stream is fully closed. Requests and responses are considered complete when the corresponding QUIC stream is closed in the appropriate direction. There's no normative language, but there's an expectation that the closure will happen at the end of the request. CONNECT is different, as noted, and (if you support CONNECT, at the very least), you have to look at the request to determine whether to expect it to be closed. In the opposite direction, we say: > Changes to the state of a request stream, including receiving a QUIC RESET_STREAM with any error code, do not affect the state of the server’s response. While this text is targeted at streams being reset or FIN'd *before* the request finishes, this could reasonably be interpreted/expanded to say that servers also cannot rely upon the FIN for finding the end of the request. I suspect that you need to process a complete request regardless of whether the stream has closed or not. -- 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/1972#issuecomment-436484647
- Re: [quicwg/base-drafts] http deadlock with strea… martinduke
- [quicwg/base-drafts] http deadlock with stream FI… martinduke
- Re: [quicwg/base-drafts] http deadlock with strea… Nick Banks
- Re: [quicwg/base-drafts] http deadlock with strea… afrind
- Re: [quicwg/base-drafts] http deadlock with strea… Kazuho Oku
- Re: [quicwg/base-drafts] http deadlock with strea… Ryan Hamilton
- Re: [quicwg/base-drafts] http deadlock with strea… Lucas Pardue
- Re: [quicwg/base-drafts] http deadlock with strea… Nick Banks
- Re: [quicwg/base-drafts] http deadlock with strea… Lucas Pardue
- Re: [quicwg/base-drafts] http deadlock with strea… MikkelFJ
- Re: [quicwg/base-drafts] http deadlock with strea… Nick Banks
- Re: [quicwg/base-drafts] http deadlock with strea… Kazuho Oku
- Re: [quicwg/base-drafts] http deadlock with strea… Mike Bishop
- Re: [quicwg/base-drafts] http deadlock with strea… martinduke
- Re: [quicwg/base-drafts] http deadlock with strea… Mike Bishop
- Re: [quicwg/base-drafts] http deadlock with strea… Dmitri Tikhonov
- Re: [quicwg/base-drafts] http deadlock with strea… David Schinazi
- Re: [quicwg/base-drafts] http deadlock with strea… Kazuho Oku
- Re: [quicwg/base-drafts] http deadlock with strea… David Schinazi
- Re: [quicwg/base-drafts] http deadlock with strea… Ryan Hamilton
- Re: [quicwg/base-drafts] http deadlock with strea… Mike Bishop
- Re: [quicwg/base-drafts] http deadlock with strea… Dmitri Tikhonov
- Re: [quicwg/base-drafts] http deadlock with strea… Ryan Hamilton
- Re: [quicwg/base-drafts] http deadlock with strea… Mike Bishop