Re: [Uuidrev] Last (hopefuly) comments on rfc4122bis

ben@yocto.com Tue, 24 October 2023 22:10 UTC

Return-Path: <ben@yocto.com>
X-Original-To: uuidrev@ietfa.amsl.com
Delivered-To: uuidrev@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 85292C14CF09 for <uuidrev@ietfa.amsl.com>; Tue, 24 Oct 2023 15:10:19 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.909
X-Spam-Level:
X-Spam-Status: No, score=-1.909 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id QdoiKVXPktTI for <uuidrev@ietfa.amsl.com>; Tue, 24 Oct 2023 15:10:17 -0700 (PDT)
Received: from mx2.yocto.eu (ns2.yoctodns.com [136.144.225.232]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3C64CC14CEF9 for <uuidrev@ietf.org>; Tue, 24 Oct 2023 15:10:16 -0700 (PDT)
MIME-Version: 1.0
Authentication-Results: mx2.yocto.eu; auth=pass smtp.mailfrom=ben@yocto.com
Date: Tue, 24 Oct 2023 22:09:41 +0000
From: ben@yocto.com
To: uuidrev@ietf.org
In-Reply-To: <PH0PR11MB5029D10DA331B19EF2A5866BBBDFA@PH0PR11MB5029.namprd11.prod.outlook.com>
References: <CAL0qLwYg9GMPPb_Vbjy4kkCWehtBv9hvkAcnOgX0yfaYpn-QgQ@mail.gmail.com> <PH0PR11MB5029AD9582D5F7C8BA0CBD15BBD8A@PH0PR11MB5029.namprd11.prod.outlook.com> <CAL0qLwametezGDeFbT4nVDHEgX2YKpe7RcuVZJ2U=vzUkoHYmw@mail.gmail.com> <PH0PR11MB5029F133F6C0EB88C7D7FE58BBD8A@PH0PR11MB5029.namprd11.prod.outlook.com> <PH0PR11MB5029D10DA331B19EF2A5866BBBDFA@PH0PR11MB5029.namprd11.prod.outlook.com>
Message-ID: <23cf1c1776d8b218cd3503dc22cf7660@yocto.com>
X-Sender: ben@yocto.com
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 8bit
X-Spamd-Bar: ---
Archived-At: <https://mailarchive.ietf.org/arch/msg/uuidrev/rJbY1YHmX5m8CDxFQrS8paF2ShM>
Subject: Re: [Uuidrev] Last (hopefuly) comments on rfc4122bis
X-BeenThere: uuidrev@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Revise Universally Unique Identifier Definitions <uuidrev.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/uuidrev>, <mailto:uuidrev-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/uuidrev/>
List-Post: <mailto:uuidrev@ietf.org>
List-Help: <mailto:uuidrev-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/uuidrev>, <mailto:uuidrev-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 24 Oct 2023 22:10:19 -0000

Hello all,

I had the following comments when revieving the latest draft of UUIDRev:

  - Section 3.2: It seems really useless to me to list abbreviations 
about 9 times. I would suggest just "UUID" and "UUIDvX, where X is the 
UUID version (1 to 8)." or something similar.

  - Section 3.2: SHA-224, SHA-256 and SHA-512. Note the difference 
between SHA-XXX and SHA3-XXX. The first one is from the SHA2 family, and 
the last one from the SHA3 family. I would suggest to change the 
description "Secure Hash Algorithm with message digest size of 224 bits" 
to "Secure Hash Algorithm 2 with message digest size of 224 bits" (just 
adding a 2, so that it is clear it is SHA2).

  - Section 3.2: UTC is actually "Universal Time Coordinated", not 
"Coordinated Universal Time".

  - Section 4: In figure 3, the word "integer" is used. Do we talk about 
signed or unsigned here? What if the first bit of an UUID is set? Does 
it get a minus sign? If not, we should add "unsigned" to the 
description, else "signed".

  - Section 4.2: Remove unused/future UUIDs, see 
https://github.com/ietf-wg-uuidrev/rfc4122bis/pull/173.

  - Section 5.5: Suggest to change "These name-based UUIDs MUST NOT 
utilize UUIDv5 and MUST be within the UUIDv8 space defined by Section 
5.8." to "These name-based UUIDs MUST NOT utilize UUIDv5 and SHOULD be 
within the UUIDv8 space defined by Section 5.8." or use something with 
"RECOMMENDED".

- Section 5.10: At a note so that people will be remembered that 
Max/Omni UUID falls outside the specified variant. For example: "Note 
that the Max/Omni UUID falls in a range of a not yet defined variant."

- Section 6.6: Is there anything confirmed about the allocation of those 
namespace ID?

- Section 7: I see "UUID Subtypes" and "UUID Namespace ID". I don't see 
"UUID Variant" and "UUID Special Forms". Is this out of scope?

Ben

Kyzer Davis (kydavis) schreef op 2023-10-24 13:38:
> Murray,
> 
> The diff for these changes can be found below:
> 
> https://github.com/ietf-wg-uuidrev/rfc4122bis/pull/173/files?diff=split&w=1
> 
> Let me know if you see anything I should change before I reupload.
> 
> Thanks,
> 
> From: Uuidrev <uuidrev-bounces@ietf.org> On Behalf Of Kyzer Davis
> (kydavis)
> Sent: Monday, October 23, 2023 1:44 PM
> To: Murray S. Kucherawy <superuser@gmail.com>
> Cc: uuidrev@ietf.org
> Subject: Re: [Uuidrev] Last (hopefuly) comments on rfc4122bis
> 
> Thanks Murray,
> 
> I have these 3 items on the tracker.
> I will iterate quickly and get this re-uploaded.
> 
> Thanks,
> 
> From: Murray S. Kucherawy <superuser@gmail.com>
> Sent: Monday, October 23, 2023 12:33 PM
> To: Kyzer Davis (kydavis) <kydavis@cisco.com>
> Cc: uuidrev@ietf.org
> Subject: Re: [Uuidrev] Last (hopefuly) comments on rfc4122bis
> 
> On Mon, Oct 23, 2023 at 7:20 AM Kyzer Davis (kydavis)
> <kydavis@cisco.com> wrote:
> 
>> Thanks Murray,
>> 
>> Let me comment on a few of these I can sus out what changes are
>> needed (if any).
>> 
>>> What are the maxima for ID and Hex ID?  Are they always the same?
>> If so, do we need both?
>> 
>> I had originally included 0 through 15 (0x0 through 0xF) but I was
>> lead to remove the “Reserved for future definition” entries.
>> 
>> This table is a delta of Table 2 in this doc which does list the
>> full list of versions available.
>> 
>> As for having both: I can drop Hex column, it is a bit superfluous.
> 
> I think the registry creation needs to say explicitly that there's a
> range of valid values.  I wouldn't expect IANA to read and understand
> the entire document.
> 
>>> Are any subtypes other than "version" possible?
>> 
>>> Are any variants other than "OSF DCE / IETF" possible?
>> 
>> Yes, technically speaking there are two non-IETF variants each with
>> own subtyping mechanism (“Family” and whatever Microsoft may
>> have in use.)
>> There is a fourth, unused, variant that could at some point in the
>> future be claimed by some other standards body and feature a
>> subtyping mechanism independent of the other three.
>> 
>> I included the “subtype” > “Version” and “Variant Name”
>> columns as a method to promote some variant agnostic extendibility
>> of this registry. Thus, at some point the page could potentially be
>> updated to include “all UUID subtypes from other variants” one
>> place.
>> 
>> However, I recognize that for the purpose of this document I can
>> only register what we have defined in this spec so I scoped the
>> actual registration to only the OSF DCE / IETF variant and subtype.
>> 
>> If I need to provide some text around these last few paragraphs I
>> can do so.
> 
> If the column is free-form, it should say that.  If there's a specific
> set of valid values, they need to be enumerated and defined.
> 
>>> The bullets at the end of this section don't read clearly.  It
>> might be enough to say "On systems ..." for each of them.
>> 
>> Sure, that is an easy enough change. If I must modify anything for
>> the aforementioned topics, I can sneak this in else I am not sure a
>> whole draft-14 just for that small change is warranted.
> 
> I'd rather send this with that clarification, since these other things
> need adjustment anyway.
> 
> -MSK