Re: [quicwg/base-drafts] Tolerate unknown stream types, add greasing (#1525)
Mike Bishop <notifications@github.com> Fri, 06 July 2018 21:42 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 A82C11310B5 for <quic-issues@ietfa.amsl.com>; Fri, 6 Jul 2018 14:42:31 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.009
X-Spam-Level:
X-Spam-Status: No, score=-8.009 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, 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, T_DKIMWL_WL_HIGH=-0.01, URIBL_BLOCKED=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 tCHYgPmIe8uG for <quic-issues@ietfa.amsl.com>; Fri, 6 Jul 2018 14:42:28 -0700 (PDT)
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 421C31310C7 for <quic-issues@ietf.org>; Fri, 6 Jul 2018 14:42:28 -0700 (PDT)
Date: Fri, 06 Jul 2018 14:42:27 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1530913347; bh=vKTsFHiWvV6A1sUAH/x6cldF54Q1SDZOA84fZsD7CvI=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=ASwlzS6E+DYZOmjjPY7fnMHtSxBWIJgVsZIph6JrGbLe42qEmdxfZQnyyPKSIcICi +VaTxuQRy49hy04ABOxvCcQxevKzviw6c8OxedYOdISw17rtp5iocEiCFyqQyVhqPS iDAzEloxwccTxBIUO6hPS+Q6inQ8QIkR827tU4vA=
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abff536f02e0f61ea4171bc94dc31ee885d86029b992cf000000011757a44392a169ce142f90e7@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1525/review/135164678@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1525@github.com>
References: <quicwg/base-drafts/pull/1525@github.com>
Subject: Re: [quicwg/base-drafts] Tolerate unknown stream types, add greasing (#1525)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b3fe24399839_59633f864ffe2f801179f8"; 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/Uc23fAmART5nuTQeiYHje_7uYTo>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.26
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, 06 Jul 2018 21:42:41 -0000
MikeBishop commented on this pull request. > If the stream header indicates a stream type which is not supported by the -recipient, this SHOULD be treated as a stream error of type -HTTP_UNKNOWN_STREAM_TYPE. The semantics of the remainder of the stream are -unknown. Implementations SHOULD NOT send stream types the peer is not already -known to support, since a stream error can be promoted to a connection error at -the peer's discretion (see {{errors}}). +recipient, the remainder of the stream cannot be consumed as the semantics are +unknown. Recipients of unknown stream types MAY trigger a QUIC STOP_SENDING +frame with an error code of HTTP_UNKNOWN_STREAM_TYPE, but MUST NOT consider such +streams to be an error of any kind. + +Implementations SHOULD NOT send stream types the peer is not already known to +support. Stream types which could modify the state or semantics of existing The recommendation is purely because if you send types that aren't supported, you're wasting bandwidth. It's not an interop issue, so perhaps that doesn't rise to the level of SHOULD NOT. Would "MAY send" be more affirming? -- 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/1525#discussion_r200776947
- Re: [quicwg/base-drafts] Tolerate unknown stream … Lucas Pardue
- Re: [quicwg/base-drafts] Tolerate unknown stream … Lucas Pardue
- Re: [quicwg/base-drafts] Tolerate unknown stream … Mike Bishop
- Re: [quicwg/base-drafts] Tolerate unknown stream … Mike Bishop
- Re: [quicwg/base-drafts] Tolerate unknown stream … Mike Bishop
- Re: [quicwg/base-drafts] Tolerate unknown stream … Mike Bishop
- Re: [quicwg/base-drafts] Tolerate unknown stream … Lucas Pardue
- Re: [quicwg/base-drafts] Tolerate unknown stream … Lucas Pardue
- Re: [quicwg/base-drafts] Tolerate unknown stream … Mike Bishop
- Re: [quicwg/base-drafts] Tolerate unknown stream … janaiyengar
- Re: [quicwg/base-drafts] Tolerate unknown stream … Mike Bishop
- Re: [quicwg/base-drafts] Tolerate unknown stream … Martin Thomson
- Re: [quicwg/base-drafts] Tolerate unknown stream … Martin Thomson
- Re: [quicwg/base-drafts] Tolerate unknown stream … Mike Bishop
- Re: [quicwg/base-drafts] Tolerate unknown stream … Lucas Pardue
- Re: [quicwg/base-drafts] Tolerate unknown stream … Mike Bishop
- [quicwg/base-drafts] Tolerate unknown stream type… Mike Bishop
- Re: [quicwg/base-drafts] Tolerate unknown stream … Martin Thomson