[quicwg/base-drafts] b40f81: Limit pushes using MAX_PUSH_ID

Martin Thomson <martin.thomson@gmail.com> Thu, 10 August 2017 00:42 UTC

Return-Path: <bounce+565321.40f-quic-issues=ietf.org@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 3E5821324D6 for <quic-issues@ietfa.amsl.com>; Wed, 9 Aug 2017 17:42:10 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.798
X-Spam-Level:
X-Spam-Status: No, score=-0.798 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_ADSP_CUSTOM_MED=0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, FREEMAIL_FORGED_FROMDOMAIN=0.199, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=0.001, NML_ADSP_CUSTOM_MED=0.9, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.com; domainkeys=pass (1024-bit key) header.sender=martin.thomson=gmail.com@github.com 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 41y0KSpR1LXD for <quic-issues@ietfa.amsl.com>; Wed, 9 Aug 2017 17:42:08 -0700 (PDT)
Received: from m69-170.mailgun.net (m69-170.mailgun.net [166.78.69.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 81792132139 for <quic-issues@ietf.org>; Wed, 9 Aug 2017 17:42:08 -0700 (PDT)
DKIM-Signature: a=rsa-sha256; v=1; c=relaxed/relaxed; d=github.com; q=dns/txt; s=mailo; t=1502325727; h=Content-Transfer-Encoding: Content-Type: Mime-Version: Subject: Message-ID: To: Reply-To: From: Date: Sender; bh=NOmNiFYHETMYXBboeVDK74snCH0r7qW60+efY/eShMY=; b=I923CcA4ZPLO53OqItjC5RqJOKDI9qJOFsoAo3ScTafdGkURvK2nsDQ3Dliv2Fm0bEmXJiEC 3q2uByDaVmTshhk/ENhOg+VCibjfxYKX6TKQyqnW86lId3iN5SNYGNrOKVIKSLksfQELUegt sGUsk9jkWcXkngL7HOe2pGJT4Qc=
DomainKey-Signature: a=rsa-sha1; c=nofws; d=github.com; s=mailo; q=dns; h=Sender: Date: From: Reply-To: To: Message-ID: Subject: Mime-Version: Content-Type: Content-Transfer-Encoding; b=oJdlvFG9fKyw0YeOXNMDvtCkG7Bk62lKzKmvZeOJ0/jLDwCwnInvD8VSnaL2T+PRV57dfv 0YgwDQUqad/ZFjK9b3/bPeKE9U56jutF8WoJnaKjdsHNBAl4yqcnmMaSY+IIDnwNcZT/7Pzo bIZL6jUgLGpN+t7acWamVEWaABsS4=
Sender: martin.thomson=gmail.com@github.com
X-Mailgun-Sending-Ip: 166.78.69.170
X-Mailgun-Sid: WyJhNzYyYiIsICJxdWljLWlzc3Vlc0BpZXRmLm9yZyIsICI0MGYiXQ==
Received: from github.com (Unknown [192.30.252.34]) by mxa.mailgun.org with ESMTP id 598babde.7fd2bc185ae0-smtp-out-n03; Thu, 10 Aug 2017 00:42:06 -0000 (UTC)
Date: Wed, 09 Aug 2017 17:42:05 -0700
From: Martin Thomson <martin.thomson@gmail.com>
Reply-To: Martin Thomson <martin.thomson@gmail.com>
To: quic-issues@ietf.org
Message-ID: <598babdddd0dc_4133f8a41933c349961b@hookshot-fe2-cp1-prd.iad.github.net.mail>
Subject: [quicwg/base-drafts] b40f81: Limit pushes using MAX_PUSH_ID
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="--==_mimepart_598babdddcd16_4133f8a41933c349956d"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/DcR3rXXsl9AV4AkkQB5jrwNrRb0>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
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: Thu, 10 Aug 2017 00:42:10 -0000

  Branch: refs/heads/push-id-seq
  Home:   https://github.com/quicwg/base-drafts
  Commit: b40f817b6bc691109491dad0bb15d6bc33b3a415
      https://github.com/quicwg/base-drafts/commit/b40f817b6bc691109491dad0bb15d6bc33b3a415
  Author: Martin Thomson <martin.thomson@gmail.com>
  Date:   2017-08-10 (Thu, 10 Aug 2017)

  Changed paths:
    M draft-ietf-quic-http.md

  Log Message:
  -----------
  Limit pushes using MAX_PUSH_ID

This repurposes SETTINGS_ENABLE_PUSH as the initial limit.

A client can use this to limit the number of outstanding pushes.  I've included
advice on increasing the limit, not as the pushes are first used, but as the
pushes are completed (or cancelled).

There wasn't a rule for handling a truncated push stream header, so I added
that as well.  The HTTP_DUPLICATE_PUSH error was reclaimed for signaling all
three types of error with a push stream header: truncation, duplicate Push ID,
and exceeding MAX_PUSH_ID.


  Commit: aa26c0f25ae24c25e4cf9241603c2fd21703d611
      https://github.com/quicwg/base-drafts/commit/aa26c0f25ae24c25e4cf9241603c2fd21703d611
  Author: Martin Thomson <martin.thomson@gmail.com>
  Date:   2017-08-10 (Thu, 10 Aug 2017)

  Changed paths:
    M draft-ietf-quic-http.md

  Log Message:
  -----------
  Fix error codes, prohibit reduction


  Commit: 636e9442eb24bcff56cdbc0eedee3ca066ce770b
      https://github.com/quicwg/base-drafts/commit/636e9442eb24bcff56cdbc0eedee3ca066ce770b
  Author: Martin Thomson <martin.thomson@gmail.com>
  Date:   2017-08-10 (Thu, 10 Aug 2017)

  Changed paths:
    M draft-ietf-quic-http.md

  Log Message:
  -----------
  Review comments


  Commit: f1fb1fec1f970e9ae99827dc0ab67cdfba0bc2af
      https://github.com/quicwg/base-drafts/commit/f1fb1fec1f970e9ae99827dc0ab67cdfba0bc2af
  Author: Martin Thomson <martin.thomson@gmail.com>
  Date:   2017-08-10 (Thu, 10 Aug 2017)

  Changed paths:
    M draft-ietf-quic-http.md

  Log Message:
  -----------
  Remove push setting in favor of MAX_PUSH_ID


  Commit: 95ff370669f9f17c8cb38835e4cf6ba7d5acdf88
      https://github.com/quicwg/base-drafts/commit/95ff370669f9f17c8cb38835e4cf6ba7d5acdf88
  Author: Martin Thomson <martin.thomson@gmail.com>
  Date:   2017-08-10 (Thu, 10 Aug 2017)

  Changed paths:
    M draft-ietf-quic-http.md

  Log Message:
  -----------
  Remove redundant word


Compare: https://github.com/quicwg/base-drafts/compare/b6bcd43af27b...95ff370669f9