[asterisk-dev] [Code Review] Add option to prevent SIP diversion headers from being sent

[Mark Michelson]

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

Ship it!


Lookin' good

- Mark


On Feb. 23, 2012, 1:13 p.m., jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1769/
> -----------------------------------------------------------
> 
> (Updated Feb. 23, 2012, 1:13 p.m.)
> 
> 
> Review request for Asterisk Developers and Mark Michelson.
> 
> 
> Summary
> -------
> 
> A while back, when 1.6.3 was a thing, A largish patch was introduced that enabled SIP diversion headers to be sent, presumably for call forwarding.  This adds an option to keep diversion headers introduced with that patch from being added to SIP dialogs because of interoperability issues that were introduced with that support.
> 
> 
> This addresses bug ASTERISK-16862.
>     https://issues.asterisk.org/jira/browse/ASTERISK-16862
> 
> 
> Diffs
> -----
> 
>   /trunk/CHANGES 355731 
>   /trunk/channels/chan_sip.c 355731 
>   /trunk/channels/sip/include/sip.h 355731 
>   /trunk/configs/sip.conf.sample 355731 
> 
> Diff: https://reviewboard.asterisk.org/r/1769/diff
> 
> 
> Testing
> -------
> 
> I actually couldn't find a useful test scenario.  Call forwarding is a pretty roughly defined feature that tends to be implemented in the dialplan, and I couldn't find a method for call forwarding that actually included the Diversion header.  Still, the approach is really simple.  I don't think it'll be a problem.
> 
> 
> Thanks,
> 
> jrose
> 
>

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