[quicwg/base-drafts] Truncated http stream error handling is aggressive (#2718)

MikkelFJ <notifications@github.com> Fri, 17 May 2019 06:55 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 []) by ietfa.amsl.com (Postfix) with ESMTP id EBFE61200A2 for <quic-issues@ietfa.amsl.com>; Thu, 16 May 2019 23:55:08 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.393
X-Spam-Status: No, score=-6.393 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, 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, RCVD_IN_MSPIKE_H2=-0.001, 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 ([]) by localhost (ietfa.amsl.com []) (amavisd-new, port 10024) with ESMTP id mGE-_QNgE_Jr for <quic-issues@ietfa.amsl.com>; Thu, 16 May 2019 23:55:07 -0700 (PDT)
Received: from out-5.smtp.github.com (out-5.smtp.github.com []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0DB56120088 for <quic-issues@ietf.org>; Thu, 16 May 2019 23:55:07 -0700 (PDT)
Date: Thu, 16 May 2019 23:55:05 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1558076105; bh=ujhch2KW6sMbeIJ4rTbrfyJ9lpurYION3Or6Mvb7myQ=; h=Date:From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post: List-Unsubscribe:From; b=0vNnjcf1YKNaRrpvfldGf6gjy/Zq2JZh0mLGggz2kqday041nOy17JKZvOQKAasbo vBUuEWkyAQ8Xc1rqf2OIBvcxL/s8MCU2u+qXspC/9bvZpgecIPIiOGwDLlzCwmXMRw gTjINGSyOmKAhqiwyfRomAuK/5h3rockW03o1Z7s=
From: MikkelFJ <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK3XS3NVWXQHCMQTXL525OGUTEVBNHHBVCU6EM@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/2718@github.com>
Subject: [quicwg/base-drafts] Truncated http stream error handling is aggressive (#2718)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5cde5ac9b82b3_22533f8aa68cd96011434cb"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: mikkelfj
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/AdBg3gD5WRRQ6z3EG6nBQF81t6U>
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: Fri, 17 May 2019 06:55:09 -0000

HTTP draft sect. 6.1

> These streams carry frames related to the request/response (see Section 4.1). When a stream terminates cleanly, if the last frame on the stream was truncated, this MUST be treated as a connection error (see HTTP_MALFORMED_FRAME in Section 8.1). 

I can see the point of always sending the header fields of a frame, but if a data frame is required to be sent in full or have the connection closed, it is hard for a server to stop a cancelled request without having the connection closed when it sends an entire file in a single data frame.

Splitting up files into multiple frames ought to be done because the length is not known in advance, not to avoid connection close.

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