[asterisk-dev] [Code Review] 3587: chan_pjsip: PJSIPNotify - strip content-length headers, add documentation.

Jonathan Rose reviewboard at asterisk.org
Tue Jun 10 10:46:32 CDT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3587/
-----------------------------------------------------------

(Updated June 10, 2014, 10:46 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and Mark Michelson.


Changes
-------

Committed in revision 415658


Repository: Asterisk


Description
-------

There was some confusion on usage of the PJSIPNotify manager command... notably in how headers are added to the notify message. This patch adds documentation explaining how to add headers and also adds some of the argument vetting from the chan_sip variant (SIPNotify).


Diffs
-----

  /branches/12/res/res_pjsip_notify.c 415411 
  /branches/12/main/message.c 415411 

Diff: https://reviewboard.asterisk.org/r/3587/diff/


Testing
-------

Send some notifies with in the documented manner.  Added Content + Content-type + Content-length variables to test how that worked.


Thanks,

Jonathan Rose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140610/5673eefe/attachment.html>


More information about the asterisk-dev mailing list