[asterisk-dev] [Code Review] SendMessage strips extension from To: Header in SIP MESSAGE

mjordan reviewboard at asterisk.org
Wed Nov 23 09:30:30 CST 2011

This is an automatically generated e-mail. To reply, visit:

Review request for Asterisk Developers.


When using the MessageSend application to send a SIP MESSAGE to a non-peer, chan_sip attempts to validate the hostname / IP Address.  In the process, it strips off the destination, and fails to add it back to the sip_pvt structure before transmission.  This patch attempts to handle valid URIs passed to chan_sip from the message core that contain either a peer to send to or a URI.  Acceptable forms from MessageSend take one of the following two forms:

sip:destination at hostname
sip:destination at ip_address

Note that more complex URIs (such as "User" <sip: blah at blah>) won't be accepted by the message core, as the message technology will assume that the user name is part of the message technology type.  That limitation is outside the scope of this issue.

This addresses bug ASTERISK-18903.


  /branches/10/channels/chan_sip.c 345977 

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


Tested with:

sip:destination at hostname
sip:destination at ip_address

All three were accepted by the MessageSend application and properly forwarded to their destinations / peers.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20111123/cb67a031/attachment.htm>

More information about the asterisk-dev mailing list