[asterisk-dev] [Code Review] Fixes for sending SIP MESSAGE outside of calls.

rmudgett reviewboard at asterisk.org
Fri Jan 20 20:09:40 CST 2012


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

(Updated Jan. 20, 2012, 8:09 p.m.)


Review request for Asterisk Developers.


Changes
-------

Addressed mmichelson's comments.


Summary (updated)
-------

Fixes done by this patch:
* Expand SIP short header names so the dialplan and other code only has to deal with the full names.

* Fix authenticate MESSAGE losing custom headers added by the MESSAGE_DATA function in the authorization attempt.

* Block user from adding standard headers that are added automatically. (To, From,...)

* Pass up better From header contents for SIP to use.  Now is in the "display-name" <URI> format expected by MessageSend.
(Note that this is a behavior change that could concievably affect some people.)

* Decrement Max-Forwards header if the user transferred it from an incoming message.

* Allow the user to override the Content-Type header contents sent by MessageSend.

* Documents what SIP expects in the MessageSend(from) parameter.


This addresses bugs ASTERISK-18917, ASTERISK-18992 and ASTERISK-19058.
    https://issues.asterisk.org/jira/browse/ASTERISK-18917
    https://issues.asterisk.org/jira/browse/ASTERISK-18992
    https://issues.asterisk.org/jira/browse/ASTERISK-19058


Diffs (updated)
-----

  /branches/10/channels/chan_sip.c 352088 
  /branches/10/channels/sip/include/sip.h 352088 
  /branches/10/main/message.c 352088 

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


Testing
-------

Updated SIP MESSAGE external tests with more cases that cover some of the items fixed.  The tests pass.


Thanks,

rmudgett

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


More information about the asterisk-dev mailing list