[asterisk-dev] [Code Review] Pimp SIP Location

[Mark Michelson]

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



/team/group/pimp_my_sip/channels/chan_gulp.c
<https://reviewboard.asterisk.org/r/2379/#comment15382>

    Use RAII_VAR for these so you don't have to clean them up all over the place.



/team/group/pimp_my_sip/channels/chan_gulp.c
<https://reviewboard.asterisk.org/r/2379/#comment15383>

    s/a well/as well/



/team/group/pimp_my_sip/res/res_sip.c
<https://reviewboard.asterisk.org/r/2379/#comment15389>

    Make the local_uri variable static.



/team/group/pimp_my_sip/res/res_sip.c
<https://reviewboard.asterisk.org/r/2379/#comment15388>

    Make this static.



/team/group/pimp_my_sip/res/res_sip/location.c
<https://reviewboard.asterisk.org/r/2379/#comment15398>

    Have you performed any tests where an endpoint name had a hyphen in it?



/team/group/pimp_my_sip/res/res_sip_endpoint_identifier_ip.c
<https://reviewboard.asterisk.org/r/2379/#comment15391>

    Add a comment here explaining why the rule here is to "deny" the addresses we expect requests to arrive from. I'm guessing this is because there is an implicit "permit all" rule for ACLs and so adding deny rules allows you not to have false positives for matching?



/team/group/pimp_my_sip/res/res_sip_session.c
<https://reviewboard.asterisk.org/r/2379/#comment15396>

    Print a warning here.


- Mark


On March 11, 2013, 7:09 a.m., Joshua Colp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2379/
> -----------------------------------------------------------
> 
> (Updated March 11, 2013, 7:09 a.m.)
> 
> 
> Review request for Asterisk Developers and Mark Michelson.
> 
> 
> Summary
> -------
> 
> Knowing where something is and how to dial it is quite important, so these changes implement the following to accomplish that:
> 
> 1. A low level API is now provided for location. It's a thin wrapper over sorcery allowing retrieval, creating, updating, and deleting of AORs/contacts. It also allows explicit contacts to be configured on the AOR itself.
> 2. A dialplan function (GULP_DIAL_CONTACTS) is now provided which creates a dial string capable of dialing all contacts on an AOR.
> 3. Dialing an endpoint, AOR, or SIP URI is now possible in dial strings.
> 4. res_sip_endpoint_identifier_ip is now fully configurable and can match IP ranges as well as individual IPs.
> 
> 
> Diffs
> -----
> 
>   /team/group/pimp_my_sip/include/asterisk/res_sip.h 382784 
>   /team/group/pimp_my_sip/channels/chan_gulp.c 382784 
>   /team/group/pimp_my_sip/include/asterisk/res_sip_session.h 382784 
>   /team/group/pimp_my_sip/res/res_sip.c 382784 
>   /team/group/pimp_my_sip/res/res_sip.exports.in 382784 
>   /team/group/pimp_my_sip/res/res_sip/location.c PRE-CREATION 
>   /team/group/pimp_my_sip/res/res_sip/sip_configuration.c 382784 
>   /team/group/pimp_my_sip/res/res_sip_endpoint_identifier_ip.c 382784 
>   /team/group/pimp_my_sip/res/res_sip_sdp_audio.c 382784 
>   /team/group/pimp_my_sip/res/res_sip_session.c 382784 
> 
> Diff: https://reviewboard.asterisk.org/r/2379/diff
> 
> 
> Testing
> -------
> 
> Tested outgoing calls galore in various combinations. Also tested incoming call matching with res_sip_endpoint_identifier_ip.
> 
> 
> Thanks,
> 
> Joshua
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130311/0c77e75c/attachment-0001.htm>