I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at . Please resolve these comments along with any other Last Call comments you may receive. Document: draft-ietf-sieve-notify-sip-message-05.txt Reviewer: Brian Carpenter Review Date: 2011-09-26 IETF LC End Date: 2011-10-05 IESG Telechat date: Summary: Almost Ready -------- Minor issue: ------------ I was briefly confused by Section 2.4. (Notify tag ":importance") It only mentions three possible values of "Priority" but RFC 3261 mentions four (the extra one is "emergency") and states that "additional values can be defined elsewhere." I think it should be noted that the :importance tag can only express a subset of the SIP Priority header semantics, and does not provide a separate category for "when life, limb, or property are in imminent danger" [RFC3261]. I realise that the direction of translation is Sieve --> SIP but one tends to expect symmetry.