Re: [quicwg/base-drafts] allow STOP_SENDING on implicitly opened streams (#2352)

MikkelFJ <notifications@github.com> Sat, 19 January 2019 10:33 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 1A9C4126DBF for <quic-issues@ietfa.amsl.com>; Sat, 19 Jan 2019 02:33:53 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -12.552
X-Spam-Level:
X-Spam-Status: No, score=-12.552 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-4.553, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_IMAGE_ONLY_32=0.001, 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 0i4j-mBH_o3L for <quic-issues@ietfa.amsl.com>; Sat, 19 Jan 2019 02:33:51 -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 738BA126C01 for <quic-issues@ietf.org>; Sat, 19 Jan 2019 02:33:51 -0800 (PST)
Date: Sat, 19 Jan 2019 02:33:50 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1547894030; bh=9RZNza0qGPYrw6p5Q+6fqINpXbENewsq0dXxAoecHjQ=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=xQZpU3xA3unqaizm7cu7iEnyL6zRQMhLQzrCFjyfJ9Oni2JpP85+kRTatjNaSqCxQ 28VrBAyENBue4Be7SDjsb5/2nPtHG13Gi83slYqDd6GE9XQGglKtWeEsnu4f7uhN6M mznhWO9lEeNwpd2h4Ycv/kiAThtRRZUjcBa/N87k=
From: MikkelFJ <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abc89316a38b845910e24a1a0b1cd3273fe26d91b592cf00000001185abf0e92a169ce17e62532@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2352/c455768522@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2352@github.com>
References: <quicwg/base-drafts/pull/2352@github.com>
Subject: Re: [quicwg/base-drafts] allow STOP_SENDING on implicitly opened streams (#2352)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c42fd0ea344a_5fe33fc4984d45bc798879"; 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/OFLhWHhwCx0OXR5hOkA1UCoPf_E>
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: Sat, 19 Jan 2019 10:33:53 -0000

>  If it was permissible to send MAX_STREAM_DATA x+4, no amount of reordering will make this a protocol violation

then you have to be able track infinite amount state because the peer controls MAX_STREAM_ID.

> If it was not permissible, and the remote peer just happened to open x+4 while the MAX_STREAM_DATA frame was in flight, then the peer got lucky this one time and the protocol violation won't be detected.

You can always get lucky, by it is sufficient to risk enforcement, and more importantly, it protects against tracking infinite state.

-- 
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/2352#issuecomment-455768522