Last Call Review of draft-saintandre-impp-call-info-02
review-saintandre-impp-call-info-02-genart-lc-davies-2013-04-24-00

Request Review of draft-saintandre-impp-call-info
Requested rev. no specific revision (document currently at 04)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2013-05-14
Requested 2013-04-17
Authors Peter Saint-Andre
Draft last updated 2013-04-24
Completed reviews Genart Last Call review of -02 by Elwyn Davies (diff)
Genart Telechat review of -03 by Elwyn Davies (diff)
Secdir Last Call review of -02 by Glen Zorn (diff)
Assignment Reviewer Elwyn Davies
State Completed
Review review-saintandre-impp-call-info-02-genart-lc-davies-2013-04-24
Reviewed rev. 02 (document currently at 04)
Review result Ready
Review completed: 2013-04-24

Review
review-saintandre-impp-call-info-02-genart-lc-davies-2013-04-24

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-saintandre-impp-call-info-02.txt
Reviewer: Elwyn Davies
Review Date: 23 April 2013
IETF LC End Date: 14 May 2013
IESG Telechat date: (if known) -



Summary: Ready.  See the generic issue with the IANA registry mentioned 


below.




Major issues: None

Minor issues: None

Nits/editorial comments:


Generic comment about SIP Header Field Parameters registry: For the 


uninitiated this registry is rather opaque.  Some parameters, such as 


the Call-Info purpose parameter for which an extra value is defined 


here, have predefined values.  However the predefined values themselves 


are not in the registry and just giving a whole RFC reference for places 


where values are defined is not very helpful. For example, in the case 


of Call-Info, the initial predefined values of purpose are buried in the 


Call-Info rule in the ABNF in Section 25.1 of RFC 3261;  also, Section 


20.9 describes the predefined values (such as "icon") as 'parameters' 


rather than values of 'purpose'.  It would probably be helpful to either 


improve the references in the registry table or actaully quote the 


possible predefined values in the table.