[asterisk-dev] [Code Review] Plenty of review done - Set channel variables with setvar= when manager user originates calls

Tilghman Lesher reviewboard at asterisk.org
Tue Sep 20 12:56:35 CDT 2011


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


I'm having a fundamental issue with this change.  When we're talking about something like the "channelvars" parameter, which emits extra details in each response, those details can be ignored, and it won't have an effect.  But the issue here is that you're modifying the parameter list, and you're not providing a method to override those presets.  Beyond this, even the name seems wrong.  Whereas channel variables in sip.conf are populated every time the channel is instantiated, this new parameter only applies to a very limited set of commands (one!) in AMI.


/trunk/Makefile
<https://reviewboard.asterisk.org/r/1412/#comment8498>

    This actually relates to OS X changes, so it should not be part of this review.


- Tilghman


On Sept. 15, 2011, 10:11 a.m., Olle E Johansson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1412/
> -----------------------------------------------------------
> 
> (Updated Sept. 15, 2011, 10:11 a.m.)
> 
> 
> Review request for Asterisk Developers and Tilghman Lesher.
> 
> 
> Summary
> -------
> 
> When the manager originates calls, there's no way to add channel variables. THis patch adds a setvar= option in manager.conf for a specific account, much like in sip.conf
> 
> This is different form variables in the originate in that these are set on every call, regardless of settings in the manager client software. Good for usage in combination with the manager event filters.
> 
> 
> This addresses bug 18143.
>     https://issues.asterisk.org/jira/browse/18143
> 
> 
> Diffs
> -----
> 
>   /trunk/CHANGES 336090 
>   /trunk/Makefile 336090 
>   /trunk/configs/manager.conf.sample 336090 
>   /trunk/include/asterisk/config.h 336090 
>   /trunk/main/config.c 336090 
>   /trunk/main/manager.c 336090 
> 
> Diff: https://reviewboard.asterisk.org/r/1412/diff
> 
> 
> Testing
> -------
> 
> In-house testing while developing. Channel variables added (proved by dumpchan() )
> 
> 
> Thanks,
> 
> Olle E
> 
>

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


More information about the asterisk-dev mailing list