Re: [Gen-art] Genart last call review of draft-ietf-regext-rdap-reverse-search-24

Mario Loffredo <mario.loffredo@iit.cnr.it> Tue, 22 August 2023 09:31 UTC

Return-Path: <mario.loffredo@iit.cnr.it>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BCE32C169535; Tue, 22 Aug 2023 02:31:24 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.906
X-Spam-Level:
X-Spam-Status: No, score=-6.906 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] 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 NT0pyVT-VdDG; Tue, 22 Aug 2023 02:31:20 -0700 (PDT)
Received: from mx5.iit.cnr.it (mx5.iit.cnr.it [146.48.58.12]) by ietfa.amsl.com (Postfix) with ESMTP id 01307C14EB17; Tue, 22 Aug 2023 02:31:18 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by mx5.iit.cnr.it (Postfix) with ESMTP id 8C541C081F; Tue, 22 Aug 2023 11:31:15 +0200 (CEST)
X-Virus-Scanned: Debian amavisd-new at mx5.iit.cnr.it
Received: from mx5.iit.cnr.it ([127.0.0.1]) by localhost (mx5.iit.cnr.it [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id W_rb2Bh69Zrk; Tue, 22 Aug 2023 11:31:11 +0200 (CEST)
X-Relay-Autenticated: yes
Message-ID: <66a02f15-bdaa-2c0d-e474-34e5a8c34a7d@iit.cnr.it>
Date: Tue, 22 Aug 2023 11:27:00 +0200
Mime-Version: 1.0
To: Susan Hares <shares@ndzh.com>, gen-art@ietf.org
Cc: draft-ietf-regext-rdap-reverse-search.all@ietf.org, last-call@ietf.org, regext@ietf.org
References: <169264888506.32313.12268327966239536039@ietfa.amsl.com>
From: Mario Loffredo <mario.loffredo@iit.cnr.it>
In-Reply-To: <169264888506.32313.12268327966239536039@ietfa.amsl.com>
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/UHAmT25RAVz9wHea6nkQGCwUtPE>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-regext-rdap-reverse-search-24
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 22 Aug 2023 09:31:24 -0000

Hi Susan,

thanks a lot for your review.

Please find my comments inline.

Il 21/08/2023 22:14, Susan Hares via Datatracker ha scritto:
> Reviewer: Susan Hares
> Review result: Ready with Nits
>
> I am the assigned Gen-ART reviewer for this draft. The General Area
> Review Team (Gen-ART) reviews all IETF documents being processed
> by the IESG for the IETF Chair.  Please treat these comments just
> like any other last call comments.
>
> For more information, please see the FAQ at
>
> <https://wiki.ietf.org/en/group/gen/GenArtFAQ>.
>
> Document: draft-ietf-regext-rdap-reverse-search-??
> Reviewer: Susan Hares
> Review Date: 2023-08-21
> IETF LC End Date: 2023-08-11
> IESG Telechat date: 2023-08-24
>
> Summary: The text is readable even for a novice in RDAP.
> I appreciated how sections 13 and 14 discussed the tension between the need for
> operational data and the need for the privacy of personal information.  It is
> important that registry operators who use this technology to provide reverse
> RDAP provide clear communication to the following groups of people: a) the
> people registering this data, b) security personnel within the registry
> operator providing the data, c) any people allowed to access the data, and d)
> other registries that may import data from this registry.
>
> I find this text to be sufficient.  I will please to see the security-DIR
> review found it ready to publish.
>
> Nits/editorial comments:
> Nits:
> Nit-#1: Section 1: It would be helpful to the naive reader to provide an IETF
> link for whois in section 1.
[ML]  OK. Looks better to move up the link to RFC 3912 to the sentence 
including "... standardized Whois capability ... ".
>
> Editorial comments: English textual comments to improve readability.
> #1 Section 1, paragraph 1
> Old:/Since RDAP consequently permits a reverse search implementation complying
> with privacy protection principles, this objection is not well-founded./
> New:/Since RDAP consequently permits a reverse search implementation complying
> with privacy protection principles, this first objection is not well-founded./
[ML] OK.
>
> #2 Section 1: paragraph 2
> Old:/The other objection to the implementation of a reverse search capability
> has been connected with its impact on server processing./ New:/The second
> objection to the implementation of a reverse search capability has been
> connected with its impact on server processing./
[ML] OK.
>
> #3, Section 1: paragraph 2
> Old: / However, the core RDAP specifications already define search queries,
> with similar processing requirements, so the distinction on which this
> objection is based is not clear./ New: /However, the core RDAP specifications
> already define search queries with similar processing requirements so the basis
> of this objection is based is not clear./
[ML] Do you mean "... so the basis of this objection is not clear" ?
>
> Section 3, paragraph 2
> Old: /All of the reverse searches defined by this document (see Section 8) have
> property names that are the same as the name of the RDAP object member that is
> the subject of the search: for example, the reverse search with the property
> name "fn" relies on the value of the "fn" member inside the jCard of an entity
> object./ New: / All of the reverse searches defined by this document (see
> Section 8) have property names that are the same as the name of the RDAP object
> member that is the subject of the search. For example, the reverse search with
> the property name "fn" relies on the value of the "fn" member inside the jCard
> of an entity object./
>
[ML] OK.


Best,

Mario

>
-- 
Dott. Mario Loffredo
Senior Technologist
Technological Unit “Digital Innovation”
Institute of Informatics and Telematics (IIT)
National Research Council (CNR)
via G. Moruzzi 1, I-56124 PISA, Italy
Phone: +39.0503153497
Web: http://www.iit.cnr.it/mario.loffredo