IETF Logo Mail Archive

Re: [quicwg/base-drafts] Modify QPACK steam and instruction terminology (#2440)

Lucas Pardue <notifications@github.com> Mon, 11 February 2019 00:07 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 5129F130E0A for <quic-issues@ietfa.amsl.com>; Sun, 10 Feb 2019 16:07:59 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8
X-Spam-Level:
X-Spam-Status: No, score=-8 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, 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 EjHN3I2XH28Y for <quic-issues@ietfa.amsl.com>; Sun, 10 Feb 2019 16:07:57 -0800 (PST)
Received: from out-7.smtp.github.com (out-7.smtp.github.com [192.30.252.198]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3B261130DCB for <quic-issues@ietf.org>; Sun, 10 Feb 2019 16:07:57 -0800 (PST)
Date: Sun, 10 Feb 2019 16:07:56 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1549843676; bh=ECr0IzChA82apBhT7lFd5e142ER1dxDAQvnEc5iGhok=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=SYpeEyxOHAx39U6KT4hfaA9PQHcC5YKWbyGnIrOADI659JC7stYGqrHSpJ5cMzMQI sD++SgJ7PI1/lY9hjtA5q8m/cDm4RVx5exSPZfep/LowxKR5EcXNLGQ2YhnVTCb7B/ vv3cBiuWEW9sKNB6XJGzbM+8G/YuTOemDKZ9IliM=
From: Lucas Pardue <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab9a98bf7ba14e8820ee2fdc80c224f02052ea3d4992cf0000000118787edc92a169ce18557f5e@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2440/review/201930881@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2440@github.com>
References: <quicwg/base-drafts/pull/2440@github.com>
Subject: Re: [quicwg/base-drafts] Modify QPACK steam and instruction terminology (#2440)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c60bcdc329ab_36213f8bed4d45b421524f"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: LPardue
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/cpZFD9AgDN0mlsT9BGnEZJR_uRg>
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: Mon, 11 Feb 2019 00:08:07 -0000

LPardue commented on this pull request.



> @@ -811,11 +819,13 @@ An encoder cannot infer from this instruction that any updates to the dynamic
 table have been received.
 
 
-## Request and Push Streams
+## Header Block Instructions
+
+HTTP/3 endpoints convert header lists to headers blocks and exchange them inside
+HEADERS and PUSH_PROMISE frames. Header block instructions reference the static
+table, or dynamic table in a particular state without modifying it.

Hmm. I think I agree with your position, which is why I have some struggle with the text as it currently stand. The changes here try to carry the existing QPACK position and text where possible. I'll take editor(s) cue on whether to broaden the changes in this PR.

AFAICT the QPACK approach was to discuss instructions and how they relate to the dynamic table. In that sense, this paragraph provides a clear statement that these instructions do not mutate.

-- 
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/2440#discussion_r255364068

v2.23.0 | Report a Bug | By Email