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

[Joshua Colp]

-----------------------------------------------------------
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>