[asterisk-dev] [Code Review] 3817: res_pjsip_notify: Add the ability for PJSIPNotify AMI command and pjsip send notify CLI command to send to a URI instead of an endpoint

Jonathan Rose reviewboard at asterisk.org
Fri Jul 18 13:32:44 CDT 2014


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

(Updated July 18, 2014, 1:32 p.m.)


Review request for Asterisk Developers, Matt Jordan and Mark Michelson.


Changes
-------

Updated review to respond to everything except for the URI string checking function... since I verified that not using it still causes crashes.


Repository: Asterisk


Description
-------

Example:

Action: PJSIPNotify
URI: sip:10.10.10.10

pjsip send notify uri <type> uri <uri> [uri...]


Adds the ability to use URIs with the PJSIP notify commands instead of endpoints. These mostly work the same as the endpoint notifications.


Diffs (updated)
-----

  /trunk/res/res_pjsip_notify.c 418868 
  /trunk/res/res_pjsip.c 418868 
  /trunk/include/asterisk/res_pjsip.h 418868 

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


Testing
-------

Used existing endpoint notification and compared it to similar notifies done against URIs.  The outgoing messages looked the same as far as the details I was concerned with went.


Thanks,

Jonathan Rose

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


More information about the asterisk-dev mailing list