[quicwg/base-drafts] Capitalize (or not) stream ID, push ID (PR #4961)
Mike Bishop <notifications@github.com> Thu, 24 February 2022 20:34 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 4BF8F3A0880
for <quic-issues@ietfa.amsl.com>; Thu, 24 Feb 2022 12:34:25 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.665
X-Spam-Level:
X-Spam-Status: No, score=-3.665 tagged_above=-999 required=5
tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.576, DKIM_SIGNED=0.1,
DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1,
HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_MSPIKE_H2=-0.001,
SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_KAM_HTML_FONT_INVALID=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 afK1-CVpOXwG for <quic-issues@ietfa.amsl.com>;
Thu, 24 Feb 2022 12:34:19 -0800 (PST)
Received: from out-24.smtp.github.com (out-24.smtp.github.com [192.30.252.207])
(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))
(No client certificate requested)
by ietfa.amsl.com (Postfix) with ESMTPS id BCCCC3A0898
for <quic-issues@ietf.org>; Thu, 24 Feb 2022 12:34:19 -0800 (PST)
Received: from github-lowworker-9d2806a.ash1-iad.github.net
(github-lowworker-9d2806a.ash1-iad.github.net [10.56.102.50])
by smtp.github.com (Postfix) with ESMTP id 209BF600BDE
for <quic-issues@ietf.org>; Thu, 24 Feb 2022 12:34:19 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1645734859;
bh=T5okVH64mjzE5RQvXj7jhHIwa4lUoI6NwfL8DBuNGrE=;
h=Date:From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post:
List-Unsubscribe:From;
b=hlqYw4NPH9v2aUZCeL8Wu6mmb41eNA6aq1VhDJHloLbPx7jKXFosc9XSsuR46tNiS
D0IuUUB3JW0WpeyzrNbO2MCNJKmLPnoGNTnM15f9ThQmoFjyXkZ831YTpFjp0ZX2J/
N5MDvDBex0pkj0uKWBzCHGSN3cSnqG/0MuajPN94=
Date: Thu, 24 Feb 2022 12:34:19 -0800
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts
<reply+AFTOJKZJHABXD7HOUYFMF4OAEPGMXEVBNHHEJB24PA@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/4961@github.com>
Subject: [quicwg/base-drafts] Capitalize (or not) stream ID, push ID (PR #4961)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_6217ebcb123f3_25f1c6fc523690";
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/0PWd5ntCtoQtPtvi_VJ2nGssiGw>
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: Thu, 24 Feb 2022 20:34:25 -0000
I believe I wound up lower-casing all instances of "stream ID" outside of diagrams since H3 has no reason to talk directly about the payloads of QUIC frames. The H3 frames mostly talk about the stream ID being encoded in the packet without giving the field a name per se. For Push ID, there are some direct references to the field, so I left a few capitalized instances -- primarily in PUSH_PROMISE and a few in GOAWAY. I would appreciate a review of the ones I *didn't* change, in case anyone things I should lower-case them as well. Fixes #4950. You can view, comment on, or merge this pull request online at: https://github.com/quicwg/base-drafts/pull/4961 -- Commit Summary -- * Lower-case "stream ID" unless referencing a protocol element * Lower-case "push ID" unless referencing a protocol element * Lower-case "stream" in non-ID case -- File Changes -- M rfc9114.md (106) -- Patch Links -- https://github.com/quicwg/base-drafts/pull/4961.patch https://github.com/quicwg/base-drafts/pull/4961.diff -- Reply to this email directly or view it on GitHub: https://github.com/quicwg/base-drafts/pull/4961 You are receiving this because you are subscribed to this thread. Message ID: <quicwg/base-drafts/pull/4961@github.com>m>
- [quicwg/base-drafts] Capitalize (or not) stream I… Mike Bishop
- Re: [quicwg/base-drafts] Capitalize (or not) stre… Martin Thomson
- Re: [quicwg/base-drafts] Capitalize (or not) stre… Mike Bishop
- Re: [quicwg/base-drafts] Capitalize (or not) stre… Mike Bishop