Re: [regext] FW: I-D Action: draft-hollenbeck-regext-rfc7483bis-00.txt

Mario Loffredo <mario.loffredo@iit.cnr.it> Thu, 07 May 2020 07:45 UTC

Return-Path: <mario.loffredo@iit.cnr.it>
X-Original-To: regext@ietfa.amsl.com
Delivered-To: regext@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C42A73A09CD for <regext@ietfa.amsl.com>; Thu, 7 May 2020 00:45:56 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 w0AnXRSXC1O0 for <regext@ietfa.amsl.com>; Thu, 7 May 2020 00:45:54 -0700 (PDT)
Received: from smtp.iit.cnr.it (mx5.iit.cnr.it [146.48.98.152]) by ietfa.amsl.com (Postfix) with ESMTP id B3A6D3A09B9 for <regext@ietf.org>; Thu, 7 May 2020 00:45:53 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by smtp.iit.cnr.it (Postfix) with ESMTP id F3A48C01EE; Thu, 7 May 2020 09:45:51 +0200 (CEST)
X-Virus-Scanned: Debian amavisd-new at mx5.iit.cnr.it
Received: from smtp.iit.cnr.it ([127.0.0.1]) by localhost (mx5.iit.cnr.it [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id DsmuU42Buof0; Thu, 7 May 2020 09:45:48 +0200 (CEST)
Received: from [192.12.193.108] (pc-loffredo.nic.it [192.12.193.108]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by smtp.iit.cnr.it (Postfix) with ESMTPSA id 024DCC00D3; Thu, 7 May 2020 09:45:47 +0200 (CEST)
To: "Hollenbeck, Scott" <shollenbeck@verisign.com>, "regext@ietf.org" <regext@ietf.org>
References: <158202847369.14106.8963334452011519309@ietfa.amsl.com> <bb1c73111e9f48ff83f8b1e454faf954@verisign.com> <bb0c899d-ac1f-7646-f45c-4b7c1955d3a1@iit.cnr.it> <76649585f0a7421c939da605251e6ed3@verisign.com>
From: Mario Loffredo <mario.loffredo@iit.cnr.it>
Message-ID: <90d78f71-6ef0-7726-63ff-a7dfdf31497c@iit.cnr.it>
Date: Thu, 7 May 2020 09:43:22 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0
MIME-Version: 1.0
In-Reply-To: <76649585f0a7421c939da605251e6ed3@verisign.com>
Content-Type: text/plain; charset=iso-8859-15; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: it
Archived-At: <https://mailarchive.ietf.org/arch/msg/regext/ownULTn-AEqOTnOsbFceDsqYcko>
Subject: Re: [regext] FW: I-D Action: draft-hollenbeck-regext-rfc7483bis-00.txt
X-BeenThere: regext@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Registration Protocols Extensions <regext.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/regext>, <mailto:regext-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/regext/>
List-Post: <mailto:regext@ietf.org>
List-Help: <mailto:regext-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/regext>, <mailto:regext-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 07 May 2020 07:45:57 -0000

Hi Scott,

please find my comments below.

Il 06/05/2020 20:50, Hollenbeck, Scott ha scritto:
> Thanks, Mario! More below.
>
>   
>
> From: Mario Loffredo <mario.loffredo@iit.cnr.it>
> Sent: Wednesday, February 19, 2020 4:17 AM
> To: Hollenbeck, Scott <shollenbeck@verisign.com>om>; regext@ietf.org
> Subject: [EXTERNAL] Re: [regext] FW: I-D Action: draft-hollenbeck-regext-rfc7483bis-00.txt
>
>   
>
> Hi Scott,
>
> here in the following my feedback.
>
>   
>
> Section 4.1.: Change "lunarNIC_level_0" with "lunarNic_level_0"
>
> [SAH] I'll change lunarNic to lunarNIC. I'm also going to change the text in 4.1 a bit to be clear that these string literals are the values that should be registered with IANA and returned in RDAP responses. There won't be any confusing mention of prefixes.
OK.
>
> Section 4.2: I would replace "on the Internet" with "on the Web"
>
> [SAH] Is there any chance that one or more of the URLs might refer to something other than an http-accessible resource? I'm not sure, so I think I'd like to keep this as "Internet" rather than "Web".

No problem. I suggested "Web" instead of "Internet" because it seemed to 
me more compliant with the definition included in RFC5988 and RFC8288.

> Section 4.2: It seems to me that, according to Section 3 of RFC5988, the members "value", "rel" and "href" are all required.
>
> [SAH] I agree, so I'll change "The "href" JSON value MUST be specified" to "The "value", "rel" and "href" JSON values MUST be specified".
OK
>
> Section 4.3: I would clearly define which members of the "notice/remark" object are required and which ones are optional by using key words described in RFC2119. Maybe the  second paragraph could be written like in the following:
>
> "  Both are arrays of objects.  Each object contains an "title"
>     string representing the title of the object, an "type"
>     string denoting a registered type of remark or notice (see
>     Section 10.2.1), an array of strings named "description" for the
>     purposes of conveying any descriptive text, and an "links"
>     array as described in Section 4.2. The
>     "description" JSON value MUST be specified. All other JSON values are
>     OPTIONAL. "
>   
> [SAH] I agree, will fix.
>   
OK
>
> Section 4.5: I would clearly define which members of the "event" object are required and which ones are optional by using the key words described in RFC2119. Maybe the  paragraph below Figure 11 could be written like in the following:
>
> "  The "events" array consists of objects, each with the following
>     members:
>   
>     o  "eventAction" -- a string denoting the reason for the event
>   
>     o  "eventActor" -- an identifier denoting the actor
>        responsible for the event
>   
>     o  "eventDate" -- a string containing the time and date the event
>        occurred.
>   
>     o  "links" -- see Section 4.2
>     
>     Both the "eventAction" and "eventDate" JSON values MUST be specified. All other JSON values are
>     OPTIONAL.  "
>   
> [SAH] I'm not sure about this one. The current text doesn't say anything about any of these values being OPTIONAL (it says "The "events" array consists of objects, each with the following members"), so, by default, I understand the text to mean that they're all REQUIRED. I could say "The "events" array consists of objects, each with the following REQUIRED members". Would that work?
All the "event" objects appearing in the doc include both the 
"eventAction" and "eventDate" members and only sometimes the remaining 
two members so we can derived that only "eventAction" and "eventDate" 
are required.
>
> Section 4.8: I would clearly define that both the members of the "publicId" object are required.
>
> [SAH] OK. I'll change "with each object containing the following members" to "with each object containing the following REQUIRED members".
OK
>
> Section 5.1: I wonder which kinds of relationships model both the entity properties "networks" and "autnums". I mean, do they model the reverse relationships between, respectively, a network or an autnum and the related entities or something else?
>
> [SAH] Maybe one of the RIR guys can address this question. Jasdip? Tom? Anyone?
>
> Section 5.2: Self link's URIs in the example should contain either the ldhName or the unicodeName. Similarly for other examples including self links to domain or nameserver objects
>
> [SAH] Agreed, will fix.
OK
>
> Section 5.2: The sentence "Figure 18 is an example of a nameserver object with all values given." seems a bit mileading to me because the example doesn't include the "entities" property. Maybe it could be written like in the following:
>
> "Figure 18 is an example of a nameserver object with nearly all the information given."
>   
> [SAH] I'll change this to "with all appropriate values given". I don't believe entities are commonly associated with name servers.
OK
>
> Section 6: Is the "description" property required in the error response ?
>
> [SAH] According to the current text, yes.
OK
>
> Section 10.2.3: Does the "transfer" event action refer to "transfer between registrars" instead of "transfer between registrants" ?
>
> [SAH] I think the intention was "between registrars", but it's also possible for objects like domains to be transferred between registrants. The current IANA registry entry says "from one registrant to another". We may need to submit an erratum for this since the IESG is the controller for values in this registry.
>
> Appendix C: I would enclose in quotes the word label in the sentence "... It uses the label attribute..."
>
> [SAH] Agreed, will fix. Thanks for the detailed review!

OK

Best,

Mario

>
> Scott
>
-- 
Dr. Mario Loffredo
Systems and Technological Development Unit
Institute of Informatics and Telematics (IIT)
National Research Council (CNR)
via G. Moruzzi 1, I-56124 PISA, Italy
Phone: +39.0503153497
Mobile: +39.3462122240
Web: http://www.iit.cnr.it/mario.loffredo
#pleasestayathome