Re: [quicwg/base-drafts] Allow provisional registrations for all registries (#3170)

Mike Bishop <notifications@github.com> Wed, 06 November 2019 19:15 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 52F02120104 for <quic-issues@ietfa.amsl.com>; Wed, 6 Nov 2019 11:15:45 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.382
X-Spam-Level:
X-Spam-Status: No, score=-6.382 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_24=1.618, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, 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 KIVCPOG-ScUw for <quic-issues@ietfa.amsl.com>; Wed, 6 Nov 2019 11:15:44 -0800 (PST)
Received: from out-19.smtp.github.com (out-19.smtp.github.com [192.30.252.202]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D413F12010C for <quic-issues@ietf.org>; Wed, 6 Nov 2019 11:15:43 -0800 (PST)
Received: from github-lowworker-c53a806.ac4-iad.github.net (github-lowworker-c53a806.ac4-iad.github.net [10.52.23.45]) by smtp.github.com (Postfix) with ESMTP id DA750520355 for <quic-issues@ietf.org>; Wed, 6 Nov 2019 11:15:42 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1573067742; bh=R0c3KxeWdCRbVszWpggz9xyDkSn8A4VRXZ12mvVT4m8=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=rBvofNhsNbveYTaoToAvDoaPOjO+zdY5FBJbuS9t3PL+z8uZuBresZVIATWRnppNZ F34S9/SBfFpJ9I/3AkfvWghFPGOdzfK1vxV/Eu4r8ob5ELBJb/Zqa0rfptgFKll8or ZdEt1ndVYazlS58AwIdXJD2nmWY72MsaFPCymfuw=
Date: Wed, 06 Nov 2019 11:15:42 -0800
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK4WWPTUDBQB3AA26ZN32BHF5EVBNHHB5KNIHE@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3170/review/312678087@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3170@github.com>
References: <quicwg/base-drafts/pull/3170@github.com>
Subject: Re: [quicwg/base-drafts] Allow provisional registrations for all registries (#3170)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5dc31bdecc39a_14bc3fa8998cd96812565a"; 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/kawJC1FM1ZqQzRmR_npx3b_ZNsY>
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: Wed, 06 Nov 2019 19:15:45 -0000

MikeBishop requested changes on this pull request.

In lots of places, you reference {{iana-policy}} for things that aren't actually described there -- it just contains a pointer to the QUIC spec for the policy.  It might be better to point directly to the QUIC spec in those places.  (Though given that the section number in QUIC could potentially change, it's also a reasonable argument that we want to have one place to update.)  Just want to make sure that's intentional, rather than a copy-paste of the policy from the QUIC registries.



-- 
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/3170#pullrequestreview-312678087