Re: [regext] Genart last call review of draft-ietf-regext-unhandled-namespaces-07

"Gould, James" <jgould@verisign.com> Wed, 10 February 2021 13:59 UTC

Return-Path: <jgould@verisign.com>
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 A3CDF3A1076; Wed, 10 Feb 2021 05:59:39 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.099
X-Spam-Level:
X-Spam-Status: No, score=-2.099 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=verisign.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 uTo58FBsz1gC; Wed, 10 Feb 2021 05:59:38 -0800 (PST)
Received: from mail6.verisign.com (mail6.verisign.com [69.58.187.32]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 5DEF13A1073; Wed, 10 Feb 2021 05:59:37 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=verisign.com; l=5920; q=dns/txt; s=VRSN; t=1612965578; h=from:to:cc:subject:date:message-id:content-id: content-transfer-encoding:mime-version; bh=NEndGnqXEuw8fHE/9H3o8pb/cdG8WX/6Q4OTOIjohmE=; b=Tw+TG5ZzlVQyp7dJtg1HEYMHXNB9Np4MBgdk6AwF/R2Aorv27iLTRVCI 3y+klNRpmg21f9M0NUv7nDCVPF4s2lDXAl5ZFuGGBtR6DIl6xPH1YftJ3 a8uS8Oe+bNdfpshB0OXyl98Tt3AOVBytzDZqwuLi/z8rXkE7sw+zKM7WQ Puy6+GdvhatO9V4Nu+mpRXjTs/gOWT9K20wyIKX+j30BEdU0XnrExEvgW lK3NovL84MxcerO0GQa8y0e/NN9nakRLzKBVOlImZYFJSNKA2PMwAG25L NmUbYuUkLzb5fxOtcUYGld05JNlLh3qDvKZPyt79gT0iABBCSZKawAez5 w==;
IronPort-SDR: CuuQDg3skxwygsTEBcEVEuD8aT5NNWtAGP8wwt56vP61DqTf6MmK2JIF6XPm5C24dB+U07/yBV Y9lw72EmcRthKumePYKnvxl6suXl/+8c03iuQf1w+u7qrzsdnuL3Ow13riNDVwuUNmmK2H5NIp gBGABEpkaTLMp2uNh08phrDgb1881ed2b7kVD9A48lWDTcbO94sjApH4i+gUp8xw0eG3ItWNVw lYNERs7LBOlK4qYX0m2RQ2vDkYg6ZXyi6CfzSrjEohhr1EbyVxvLHyrhR7qJM9UMt+t0vfLfBK 7tM=
X-IronPort-AV: E=Sophos;i="5.81,168,1610427600"; d="scan'208";a="5134535"
Received: from BRN1WNEX01.vcorp.ad.vrsn.com (10.173.153.48) by BRN1WNEX01.vcorp.ad.vrsn.com (10.173.153.48) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Wed, 10 Feb 2021 08:59:35 -0500
Received: from BRN1WNEX01.vcorp.ad.vrsn.com ([fe80::a89b:32d6:b967:337d]) by BRN1WNEX01.vcorp.ad.vrsn.com ([fe80::a89b:32d6:b967:337d%4]) with mapi id 15.01.2106.006; Wed, 10 Feb 2021 08:59:35 -0500
From: "Gould, James" <jgould@verisign.com>
To: "peter@akayla.com" <peter@akayla.com>, "gen-art@ietf.org" <gen-art@ietf.org>
CC: "draft-ietf-regext-unhandled-namespaces.all@ietf.org" <draft-ietf-regext-unhandled-namespaces.all@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>, "regext@ietf.org" <regext@ietf.org>
Thread-Topic: Genart last call review of draft-ietf-regext-unhandled-namespaces-07
Thread-Index: AQHW/7T3iPEWQzTFD06OsyEgOcrJWA==
Date: Wed, 10 Feb 2021 13:59:35 +0000
Message-ID: <415E3373-7771-4FE1-B72C-B28A1511006F@verisign.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/16.41.20091302
x-originating-ip: [10.170.148.18]
Content-Type: text/plain; charset="utf-8"
Content-ID: <7B25CDC55DA80B4FB54B339BB031701A@verisign.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/regext/Z7aqu5Hw9TM2pMXHBR2WRwJD8iY>
Subject: Re: [regext] Genart last call review of draft-ietf-regext-unhandled-namespaces-07
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: Wed, 10 Feb 2021 13:59:40 -0000

Peter,

Thank you for your review and feedback.  I provide responses to your feedback embedded below.  The updates based on your feedback and other feedback received will be included in draft-ietf-regext-unhandled-namespaces-08.  

-- 
 
JG



James Gould
Fellow Engineer
jgould@Verisign.com <applewebdata://13890C55-AAE8-4BF3-A6CE-B4BA42740803/jgould@Verisign.com>

703-948-3271
12061 Bluemont Way
Reston, VA 20190

Verisign.com <http://verisigninc.com/>

On 2/9/21, 11:49 PM, "Peter Yee via Datatracker" <noreply@ietf.org> wrote:

    Reviewer: Peter Yee
    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://secure-web.cisco.com/18XJ21QUB_6pM8xC9AxgAF1lQGdVTQGVD3ttr64Abh4xtYEHewGl5EW-GJTbRbKJMuPV8KyCHx1maHQo1jcsWHiNDDCjzavgOvt7VfmB_DlWdSxxhJXOvjbAwge8wZIjdMPCAq5-if9dJdaBbleGZzdSxIhW0jK8ZHx78azgsy9giuHdjzxHH2_RuqllFCneH9ssvSaqyoF-hnGcZykWhn56qLfTatUWQEhL4KRkUvw0jSIB3S5LnrX7UcsJWrlEd/https%3A%2F%2Ftrac.ietf.org%2Ftrac%2Fgen%2Fwiki%2FGenArtfaq>.

    Document: draft-ietf-regext-unhandled-namespaces-07
    Reviewer: Peter Yee
    Review Date: 2021-02-09
    IETF LC End Date: 2021-02-09
    IESG Telechat date: 2021-02-18

    Summary: This EPP draft specifies a means to send information about unhandled
    namespace (a service that the client or server isn't prepared to handle) by
    means of reusing <extValue>. To the extent of my limited knowledge of EPP, this
    draft is ready with nits.

    Major issues: None

    Minor issues: None

    Nits/editorial comments:

    General:

    I'm not totally taken with the term "template" for XML examples that aren't
    wholly templates. On the other hand, I'd like to think that any implementers of
    EPP would recognize which parts were truly template-like and which parts are
    borrowed from the various EPP RFCs for example.

JG - Correct, those that are aware of EPP should be able to pick-up on the use of the placeholder content variables.  In the section 1.1 "Conventions Used in This Document", we did define the placeholder content variables used in the template XML for clarity.  

    A few of the XML examples do not indent the urn in the <reason> block. While
    that shouldn't matter for the meaning or parsing, the indentation is done
    inconsistently. If this was intentional (e.g., to prevent wrapping of long
    lines), then leave it as is. While I don't think of the lines were longer than
    allowed even if two spaces were inserted before "urn", the easier visual
    parsing would be appreciated.

JG - The instances that you're referring to are located in section 6, where adding a space for the "urn:ietf:params:xml:ns:changePoll-1.0" unhandled namespace results in "Warning: Artwork too wide, reducing indentation from 3 to 2".  The "urn:ietf:params:xml:ns:domain-1.0" unhandled namespace can have two spaces added, but that would be inconsistent with the "urn:ietf:params:xml:ns:changePoll-1.0" unhandled namespace in the same example.  I'll go ahead and add the two spaces to the "urn:ietf:params:xml:ns:domain-1.0" unhandled namespace.

    Specific:

    Page 6, 1st paragraph following XML, 1st phrase: append "for an" after
    "Template".

JG - Done 

    Page 6, 1st paragraph following second block of XML, 2nd sentence: insert "an"
    before "example of".

JG - Done

    Page 7, 1st phrase: change the period to a colon to be similar to RFC 5730's
    style for examples.

JG - Done

    Page 7, section 3.2, 1st paragraph, 2nd sentence: change "a" to "an" before
    "[RFC5730] <extValue>".

JG - Done

    Page 9, 1st phrase: change the period to a colon to be similar to RFC 5730's
    style for examples.

JG - Done

    Page 10, section 4, 1st sentence: insert "a" before "new". Insert "rather
    specifies" before "an operational".  Or something similar.

JG - Done.  I used the "rather specifies".  

    Page 16, item 3, 1st sentence: consider deleting the comma after "EPP
    responses".

JG - Done