[asterisk-dev] [Code Review]: AMI version 1.4 Specification Review Request

Sean Bright reviewboard at asterisk.org
Tue Jan 8 12:45:25 CST 2013



> On Jan. 8, 2013, 9:11 a.m., elguero wrote:
> > Based on a quick look, I like it! The documentation is very clearly written.
> > 
> > Just a few thoughts:
> > 
> > Tags - My first thought was that multiple tags should not be on separate lines, like in the example provided for sip peers in sip.conf.  Generally, one would think that the first tag would be overwritten by the second tag.  What about a comma delimited list and then we could call that setting "tags" instead of "tag" (tags=special_endpoints,endpoint_group_one).  I can just picture somebody putting tag towards the top of a peer definition and then another one later on and not figuring out that they have multiple tags and wondering why they are getting messages that they shouldn't be getting.  Just throwing that out there.
> > 
> > Keys - Keys are case insensitive.  Why not case sensitive?
> > 
> > Port and bindaddr - Why don't we combine these to one setting, bindaddr?  We are doing that for tlsbindaddr and I believe other parts of Asterisk do this now too.

> Keys - Keys are case insensitive.  Why not case sensitive?

I tend to agree.  I would always err on the side of being too strict, rather than accommodating, especially when we aren't dealing with direct user input (a person typing at a keyboard).


- Sean


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


On Jan. 8, 2013, 8:41 a.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2269/
> -----------------------------------------------------------
> 
> (Updated Jan. 8, 2013, 8:41 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> A proposed revamp of the AMI protocol has been written and is available for discussion:
> 
> https://wiki.asterisk.org/wiki/display/AST/AMI+1.4+Specification
> 
> Please note that this will change the AMI protocol significantly, and represents a major shift in how the protocol represents operations in Asterisk to consumers of the protocol.
> 
> 
> Diffs
> -----
> 
> 
> Diff: https://reviewboard.asterisk.org/r/2269/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Matt
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130108/1574426a/attachment.htm>


More information about the asterisk-dev mailing list