Secdir last call review of draft-ietf-regext-bundling-registration-09

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reviewer: Russ Housley
Review result: Has Issues

I reviewed this document as part of the Security Directorate's ongoing
effort to review all IETF documents being processed by the IESG.  These
comments were written primarily for the benefit of the Security Area
Directors.  Document authors, document editors, and WG chairs should
treat these comments just like any other IETF Last Call comments.

Document: draft-ietf-regext-bundling-registration-09
Reviewer: Russ Housley
Review Date: 2019-03-09
IETF LC End Date: 2019-03-15
IESG Telechat date: unknown

Summary: Has Issues


Major Concerns:

If this document is going to be published on the IETF Stream, then the
IANA registrations should point to the IESG, not the document authors.


Minor Concerns:

Section 2: Please update the first paragraph to reference RFC 8174
in addition to RFC 2119, as follows: 

   The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT",
   "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and
   "OPTIONAL" in this document are to be interpreted as described in
   BCP 14 [RFC2119] [RFC8174] when, and only when, they appear in all
   capitals, as shown here.


Nits:

The document has many grammer errors.  For example, the first paragraph
of the Introduction has three things that need to be corrected:
- s/which has identical/which have identical/
- s/labels(LABEL)/labels (LABEL)/
- s/(V-tld);/(V-tld)./

In addition, there are several places in the document with arbitrary
extra white space.  For example, in Section 8, it says:
      <!--
      Child elements of the <b-dn:create>     command
      All     elements must be present at     time of creation
      -->

Please correct the grammar and eliminate the extra white space.

Ins Section 7.2.2, some of the lines in the examples do not begin
with "S:".  Please correct.





[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Mhonarc]     [Fedora Users]

  Powered by Linux