[quicwg/base-drafts] Retain use of SETTINGS_ENABLE_PUSH (#718)

janaiyengar <notifications@github.com> Thu, 10 August 2017 01:02 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 B77981324D7 for <quic-issues@ietfa.amsl.com>; Wed, 9 Aug 2017 18:02:29 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.181
X-Spam-Level:
X-Spam-Status: No, score=-8.181 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, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H2=-2.8, SPF_PASS=-0.001, 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 o2LXqJJ7cyKO for <quic-issues@ietfa.amsl.com>; Wed, 9 Aug 2017 18:02:27 -0700 (PDT)
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2-ext5.iad.github.net [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 1B7071324D8 for <quic-issues@ietf.org>; Wed, 9 Aug 2017 18:02:27 -0700 (PDT)
Date: Wed, 09 Aug 2017 18:02:26 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1502326946; bh=EuyNVNjQ7XhMHKrWgBVPClKQ+6fYZL9HJ4vRPz3v95k=; h=From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post: List-Unsubscribe:From; b=zrMw1uzzlAsBPXY7CEXH5sPmwJ/JD7Y6N2huUualO1KoZaB1WdFUnQdwSlEBacWS6 Hafk1+sBfKqDDuSHaQlTF1klLp01SFOSRKDtg3HojUcMCtwKo2mh19WUDBQdOEDQ77 IlQSM/JfII3+/IjD/0E6Htq5Nb6gQv7M5Kl16Gao=
From: janaiyengar <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abb67441ca4563261fa34a0b8f07abe46462cfe16d92cf0000000115a372a292a169ce0eda8cd0@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/718@github.com>
Subject: [quicwg/base-drafts] Retain use of SETTINGS_ENABLE_PUSH (#718)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_598bb0a249246_277153fc1afaf5c341620a1"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: janaiyengar
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/xuZLfvbFBSdaoLWpN6Aqf5xb7S4>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.22
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 01:02:30 -0000

When pushing, the server currently only pushes after it's heard a MAX_PUSH_ID from the client. It might be useful to retain HTTP/2's SETTINGS_ENABLE_PUSH boolean that indicates push is disabled by the client. Without this boolean, the server has to wait forever to push but can't because it didn't hear anything from the client "yet", and the only way it knows that the client has disabled push is basically silence... which seems not great. (Tertiary point: it might be useful for various debugging/logging purposes to have an explicit disabling.)

-- 
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/718