[asterisk-dev] [Code Review] SIP register not using peer's outbound proxy

Mark Michelson mmichelson at digium.com
Fri Jul 10 16:06:09 CDT 2009


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

Ship it!


Just take care of the one thing I mentioned below and I'll be all right with this change.

You may also want to document in the code the reason why the [peername?] is not being documented in sip.conf


/trunk/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/294/#comment2302>

    Any reason why you used strrchr instead of strchr? In my mind, strchr seems more appropriate.


- Mark


On 2009-07-10 15:48:22, David Vossel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/294/
> -----------------------------------------------------------
> 
> (Updated 2009-07-10 15:48:22)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> If callbackextension is defined for a peer it successfully causes a registration to occur, but the registration ignores the outboundproxy settings for the peer.  This is because the call to obproxy_get() in transmit_register() is not capable of passing a peer.  This patch adds the peername option, [peername?], to the register string allowing transmit_register to attempt to find the peer and use it's outbound proxy.
> 
> register => [peername?][transport://]user[@domain][:secret[:authuser]]@host[:port][/extension][~expiry]
> 
> 
> This addresses bug 14344.
>     https://issues.asterisk.org/view.php?id=14344
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/chan_sip.c 205840 
> 
> Diff: https://reviewboard.asterisk.org/r/294/diff
> 
> 
> Testing
> -------
> 
> tested register parsing and verified peer proxy is used over global proxy when [peername?] is specified.
> 
> 
> Thanks,
> 
> David
> 
>




More information about the asterisk-dev mailing list