[asterisk-dev] [Code Review] 3391: ARI: Don't complain about missing ARI users when we aren't enabled

Joshua Colp reviewboard at asterisk.org
Tue Mar 25 09:47:03 CDT 2014


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

Ship it!


Ship It!

- Joshua Colp


On March 25, 2014, 11:33 a.m., Sean Bright wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3391/
> -----------------------------------------------------------
> 
> (Updated March 25, 2014, 11:33 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> Currently, if ARI is not enabled it will still complain that there are no configured users.  This patch checks to see if ARI is enabled before logging and error or iterating the container to validate the users.
> 
> 
> Diffs
> -----
> 
>   /branches/12/res/ari/config.c 411084 
> 
> Diff: https://reviewboard.asterisk.org/r/3391/diff/
> 
> 
> Testing
> -------
> 
> Basic runtime tests with ARI enabled and disabled.
> 
> 
> Thanks,
> 
> Sean Bright
> 
>

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


More information about the asterisk-dev mailing list