[asterisk-bugs] [JIRA] (ASTERISK-25678) app_confbridge: Add list concise command

Corey Farrell (JIRA) noreply at issues.asterisk.org
Fri Dec 15 11:06:07 CST 2017


     [ https://issues.asterisk.org/jira/browse/ASTERISK-25678?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Corey Farrell updated ASTERISK-25678:
-------------------------------------

    Assignee: jaredmauch
      Status: Waiting for Feedback  (was: Open)

I agree with Richard, CLI is not meant for consumption by other programs.  I would support adding attributes to AMI/ARI responses but I'm -1 on addition of concise CLI commands.  It bloats CLI code and adds little value.

If you'd like to submit a change to gerrit adding the attributes you require to AMI/ARI list actions I can help you through the process.

----

Thanks for the contribution! If you'd like your contribution to be included faster, you should submit your patch for code review by the Asterisk Developer Community. To do so, please follow the Code Review \[1\] instructions on the wiki. Be sure to:
* Verify that your patch conforms to the Coding Guidelines \[2\]
* Review the Code Review Checklist \[3\] for common items reviewers will look for
* If necessary, provide tests for the Asterisk Test Suite that verify the correctness of your patch \[4\]

When ready, submit your patch and any tests to Gerrit \[5\] for code review.

Thanks!

\[1\] https://wiki.asterisk.org/wiki/display/AST/Code+Review
\[2\] https://wiki.asterisk.org/wiki/display/AST/Coding+Guidelines
\[3\] https://wiki.asterisk.org/wiki/display/AST/Code+Review+Checklist
\[4\] https://wiki.asterisk.org/wiki/display/AST/Asterisk+Test+Suite+Documentation
\[5\] https://wiki.asterisk.org/wiki/display/AST/Gerrit+Usage

> app_confbridge: Add list concise command
> ----------------------------------------
>
>                 Key: ASTERISK-25678
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-25678
>             Project: Asterisk
>          Issue Type: Improvement
>      Security Level: None
>          Components: Applications/app_confbridge
>            Reporter: jaredmauch
>            Assignee: jaredmauch
>            Severity: Minor
>         Attachments: app_confbridge.concise.patch, confbridge.patch.2.txt
>
>
> In preparing to migrate from MeetMe to Confbridge we needed some "comfort" capabilities to emulate some of the meetme cli commands used.
> This introduces a confbridge list 1234 concise style command that is helpful for machines to read the input.  This is mostly prototype code that allows us to see the talker in the CLI (for hunting noisy lines).
> I'm not wedded to anything specific except two things:
> 1) I prefer the channel name to be first
> 2) I need the talker information, without tracking all the event state.
> My code certainly isn't perfect but I've tested it and it works.



--
This message was sent by Atlassian JIRA
(v6.2#6252)



More information about the asterisk-bugs mailing list