[asterisk-dev] [Code Review] 3442: chan_sip: Add TELPHONECONTEXT channel variable for Request TEL URIs

[Matt Jordan]

> On April 12, 2014, 1:21 a.m., Olle E Johansson wrote:
> > /trunk/channels/chan_sip.c, line 8262
> > <https://reviewboard.asterisk.org/r/3442/diff/1/?file=57272#file57272line8262>
> >
> >     Sadly, I think we should name this SIPURIPHONECONTEXT or something that starts with SIP like SIPDOMAIN SIPFROMADDRESS et al. Vanity tags may be a good thing.

Works for me - changed.


- Matt


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


On April 11, 2014, 9:35 p.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3442/
> -----------------------------------------------------------
> 
> (Updated April 11, 2014, 9:35 p.m.)
> 
> 
> Review request for Asterisk Developers, Geert Van Pamel and Olle E Johansson.
> 
> 
> Bugs: ASTERISK-17179
>     https://issues.asterisk.org/jira/browse/ASTERISK-17179
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This patch is a continuation of 3349 (https://reviewboard.asterisk.org/r/3349/)
> 
> It resolves a finding oej had that the phone-context be available in a TELPHONECONTEXT channel variable. This patch adds that variable. It also allows a local number (or global number specified in the TEL URI) to be used to look up as a peer. Granted; not super useful. But hey.
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/sip/include/sip.h 412292 
>   /trunk/channels/chan_sip.c 412292 
> 
> Diff: https://reviewboard.asterisk.org/r/3442/diff/
> 
> 
> Testing
> -------
> 
> See review https://reviewboard.asterisk.org/r/3441
> 
> 
> Thanks,
> 
> Matt Jordan
> 
>

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