[Home]

Summary:ASTERISK-10312: [patch] Two missing manager command
Reporter:Tilghman Lesher (tilghman)Labels:
Date Opened:2007-09-16 09:57:39Date Closed:2007-09-22 08:45:15
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20070914__twomissingmanagercommands.diff.txt
Description:Adding the Reload event and the MeetmeList event

****** ADDITIONAL INFORMATION ******

This is part of my drive to have Command privilege become unnecessary for management, to avoid giving out the ability to remotely kill the system to anybody who wants to run a third party management interface.
Comments:By: Russell Bryant (russell) 2007-09-16 23:54:59

looks good

By: Brandon Kruse (bkruse) 2007-09-17 15:50:27

I cannot contribute to Code Review, so I figured I would test it.

No warnings/errors:

Both manager commands function, and return the correct output as expected. This includes the success response (important for any program manipulating the manager response for error checking, etc.)

-bk

By: Clod Patry (junky) 2007-09-20 22:31:05

what about MeetmeKick, MeetmeLock, MeetmeUnlock?

By: Tilghman Lesher (tilghman) 2007-09-20 22:39:15

junky:  which clients which use the manager interface use those commands?

By: Clod Patry (junky) 2007-09-20 22:43:27

one big customer I have is doing these commands many times during a day.

It's a conference service, so I think this is a must for what we really want.

By: Digium Subversion (svnbot) 2007-09-22 08:45:15

Repository: asterisk
Revision: 83532

------------------------------------------------------------------------
r83532 | tilghman | 2007-09-22 08:45:14 -0500 (Sat, 22 Sep 2007) | 2 lines

Add the MeetmeList and Reload manager commands, which supplement the need to have Command privilege. (closes issue ASTERISK-10312)

------------------------------------------------------------------------