[asterisk-dev] [Code Review] Dialplan function for manager account checks - AMI_CLIENT()

[Olle E Johansson]

> On 2009-11-03 15:50:16, Russell Bryant wrote:
> > /trunk/main/manager.c, lines 5387-5397
> > <https://reviewboard.asterisk.org/r/421/diff/1/?file=7272#file7272line5387>
> >
> >     Since the operation being performed on each session is so small, it would actually be more efficient to implement this as an ao2_callback().
> 
> Olle E Johansson wrote:
>     I take that as a suggestion, right. I just copied this code from another place in the file.

Reminder sent to Russell - I need a template to copy from...


- Olle E


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


On 2009-11-03 14:48:56, Olle E Johansson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/421/
> -----------------------------------------------------------
> 
> (Updated 2009-11-03 14:48:56)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Implementation of a dialplan function for checking manager accounts. I've made it extensible so that we can add other parameters when needed. Right now it only checks the number of logged in sessions for a manager account.
> 
> 
> Diffs
> -----
> 
>   /trunk/main/manager.c 227348 
> 
> Diff: https://reviewboard.asterisk.org/r/421/diff
> 
> 
> Testing
> -------
> 
> Tested on my Linux system and it reports manager logins properly - 0, 1 and 2 concurrent sessions...
> 
> 
> Thanks,
> 
> Olle E
> 
>