Re: [Gen-art] Gen-ART review of draft-green-secsh-ecc-08.txt

[Douglas Stebila <[e-mail]>]

Thanks.  I will make the suggested changes.

Douglas

On 2009-Jul-14, at 8:00 PM, Miguel A. Garcia wrote:

> 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 <[e-mail]>
> 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
> [phone]
> Ericsson Spain
>