[asterisk-dev] [Code Review] CLI permissions

[Eliel Sardañons]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/11/
-----------------------------------------------------------

(Updated 2008-11-28 13:05:36.768535)


Review request for Asterisk Developers.


Changes
-------

Sorry for all this updates to the review board:
- Make regular expression matching case insensitive.
- Explain that we can use regular expressions inside the configuration sample.


Summary
-------

Implement CLI permissions related to issue 11123.
I would like to receive an architectural review, if this is the right way to implement CLI permissions. I know there are some minor changes that need to be done (and others not so minor), maybe use RWLIST instead of a LIST, also trailing spaces, and a lot more, but If the architecture is ok, I will continue improving the code.

Thanks in advanced!


Diffs (updated)
-----

  /trunk/CHANGES 159772 
  /trunk/configs/cli_permissions.conf.sample PRE-CREATION 
  /trunk/configure.ac 159772 
  /trunk/include/asterisk/_private.h 159772 
  /trunk/include/asterisk/autoconfig.h.in 159772 
  /trunk/include/asterisk/cli.h 159772 
  /trunk/main/asterisk.c 159772 
  /trunk/main/cli.c 159772 

Diff: http://reviewboard.digium.com/r/11/diff


Testing
-------


Thanks,

Eliel