Skip to main content

Last Call Review of draft-ietf-geopriv-dhcp-lbyr-uri-option-
review-ietf-geopriv-dhcp-lbyr-uri-option-genart-lc-melnikov-2012-06-19-00

Request Review of draft-ietf-geopriv-dhcp-lbyr-uri-option
Requested revision No specific revision (document currently at 19)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2012-06-14
Requested 2012-06-07
Authors James Polk
I-D last updated 2012-06-19
Completed reviews Genart Last Call review of -?? by Alexey Melnikov
Genart Telechat review of -17 by Alexey Melnikov (diff)
Secdir Early review of -?? by Chris M. Lonvick
Secdir Telechat review of -17 by Chris M. Lonvick (diff)
Assignment Reviewer Alexey Melnikov
State Completed
Request Last Call review on draft-ietf-geopriv-dhcp-lbyr-uri-option by General Area Review Team (Gen-ART) Assigned
Completed 2012-06-19
review-ietf-geopriv-dhcp-lbyr-uri-option-genart-lc-melnikov-2012-06-19-00
I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
<

http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-geopriv-dhcp-lbyr-uri-option-15
Reviewer: Alexey Melnikov
Review Date: 2012-06-09
IETF LC End Date: 2012-06-14
IESG Telechat date: (if known)



Summary: This document is nearly ready for publication as a Proposed 


Standard.




Major issues: None

Minor issues:



In Section 1: the first reference to URI needs a Normative reference. 


The whole document is about passing URIs and they are not defined anywhere.






In Section 2.3: LuriType =1 description says that the value is UTF-8. 


This is not correct, URIs never contain non-ASCII values. So please 


either say ASCII, or use IRI if you really meant UTF-8 (but use of IRIs 


would be horrible, so don't).




LuriType=2: in network byte order?

In Section 3.2: "data:" URI needs an Informative reference (RFC 2397).

In Section 3.3: all media types mentioned (two?) need Normative references.


Nits/editorial comments:

In Section 3, 2nd to the last para: s/deference/dereference ?