Re: [apps-discuss] [Uri-review] New version of tn3270 draft

[Julian Reschke <[e-mail]>]

On 28.11.2010 11:06, Mykyta Yevstifeyev wrote:
> Hello all,
>
> A new version of tn3270 draft (-05) is now available on
> the I-D repository. It includes the following changes:
>
> -added the template for IANA to register the scheme as
> *Provisional*;
> -added more detailed description of TN3270 service
> (with references to corresponding RFCs);
> -added more references;
> -added Acknowledgments section;
> -another minor changes.
>
> I am sending this message to Applications Area ADs -
> Alexey Melnikov and Peter Saint-Andre in order they
> can review this document.
>
> Any suggestions for improvements are welcome.
>
> Best regards,
> Mykyta Yevstifeyev

Sunday morning nitpicking:

1) It's really confusing when you submit 3 drafts on the same day, and 
some of them only fix minor typos, or punctuation. That makes it hard to 
find out what actually changed. Also, draft -04 seems to have the wrong 
submission date.

2) A URI scheme definition that cites RFC 3986 only for security 
considerations makes me nervous. For instance, it should be clear how 
the individual components would be recognized by a generic URI parser. 
It appears that all you're saying that the scheme-specific part maps to:

   "//" authority

3) ABNF comments

3a) What's the delimiter between password and host?

3b) What's the [/] at the end for? Did you mean [ "/" ]?

3c) Please avoid prose constructions; they are not sufficient to 
actually parse the URI. So which characters are allowed in "user", 
"password" and "host"?

4) You're not citing the URI scheme registration RFC (4395), nor have 
the information requested in 
<http://tools.ietf.org/html/rfc4395#section-5.4>.

Best regards, Julian