I have been selected as the General Area Review Team (Gen-ART)
reviewer for this draft (for background on Gen-ART, please see
http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html).
Please wait for direction from your document shepherd
or AD before posting a new version of the draft.
Document: draft-green-secsh-ecc-08.txt
Reviewer: Miguel Garcia <[email protected]>
Review Date: 2009-07-14
IETF LC End Date: 2009-08-10
Summary: The document is ready for publication as a standard track RFC.
Major issues: none
Minor issues:
- In Section 9.1, second sentence, the text reads: "These curve should
always be enabled ...." I think the "should" should be elevated to a
"SHOULD" in this case.
- The IANA Considerations section needs more work. In particular, it does
not name the registry where IANA has to operate. I am assuming that this
registry is "Secure Shell (SSH) Protocol Parameters). That registry
includes several subregistries, of which, this draft is presumably adding
new values to the "Public Key Algorithm Names" and "Key Exchange Method
Names". Please clearly explain what needs to be registered where.
Nits/editorial comments: A couple of minor nits, in case you need to
create a new version of the document
- The third paragraph of Section 6.1 contains an 8-line long sentence. I
think this sentence could be simplified and chunked into smaller
sentences, because it is a bit hard to read such long sentences.
- The first paragraph in Section 7 reads ... "... without being granted
IANA permission". If we are accurate, IANA does not grant any permission.
Instead IANA acts as a registry. So, most likely you want to say "...
without requiring an IANA registration process" or something like that.
- Title of Section 9.2, missing white space separated two words.
Regards,
Miguel
--
Miguel A. Garcia
+34-91-339-3608
Ericsson Spain
_______________________________________________
Gen-art mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/gen-art
