Document: draft-ietf-sip-ipv6-abnf-fix-05 Reviewer: Suresh Krishnan Review Date: 2010/05/17 IESG Telechat date: 2010/05/20 Summary: This document is ready for publication as a Proposed Standard. I do have one comment that you may like to address. Minor ===== * The draft also fixes the ABNF for IPv4 addresses i.e. the rule but is silent about why it is doing so. I think I understand why it is being fixed but I would like the authors to clarify. The old rule read IPv4address = 1*3DIGIT "." 1*3DIGIT "." 1*3DIGIT "." 1*3DIGIT and the new rule reads IPv4address = dec-octet "." dec-octet "." dec-octet "." dec-octet \ dec-octet = DIGIT ; 0-9 / %x31-39 DIGIT ; 10-99 / "1" 2DIGIT ; 100-199 / "2" %x30-34 DIGIT ; 200-249 / "25" %x30-35 ; 250-255 It would be nice if you can some explanatory text in Section 2. Something like "The rule in RFC3261 allowed invalid IPv4 addresses to be specified and still be compliant to the ABNF. (e.g. An URI such as sip:bob@444.555.666.777:88 would still be considered valid.) The resolution for this error is simply to use the correct ABNF for the production rule from Appendix A of RFC3986 [3]."