|Summary:||ASTERISK-03610: [patch] New manager commands for SIP (Not XML any more)|
|Reporter:||Olle Johansson (oej)||Labels:|
|Date Opened:||2005-03-01 08:05:56.000-0600||Date Closed:||2008-01-15 15:28:03.000-0600|
|Environment:||Attachments:||( 0) managerpeers.txt|
( 1) managerpeers.txt
( 2) xmlpeers.txt
|Description:||This patch adds|
* SIPpeers manager command
* SIPshowpeer manager command
Output in text or XML format.
Also changes some text parsing in handle_request in order to avoid all those strcmps.
Dislaimer on file
****** ADDITIONAL INFORMATION ******
The XML model is experimental until I've implemented this for IAX and possibly other commands as well. I don't want different names for the same thing in various parts of Asterisk, so I want to be able to change. This is documented in the source as well as in the help text for the manager commands.
The manager commands will report in XMl if the
header is present in the manager request.
|Comments:||By: twisted (twisted) 2005-03-01 09:52:37.000-0600|
/me chokes on xml
Anyway, I guess its ok?
By: Olle Johansson (oej) 2005-03-01 10:12:31.000-0600
Sorry, did not mean to offend anyone :-)
XML is better from a machine point of view than the stripped printf's.
We can easily add a format like application/csv and print comma-separated data.
By: twisted (twisted) 2005-03-01 10:36:56.000-0600
hahaha... Nah, I'm not offended, just trying to bring a bit-o-humour to the bugtracker.
By: Olle Johansson (oej) 2005-03-01 14:23:41.000-0600
Might also be an option... :-)
By: twisted (twisted) 2005-03-01 15:25:25.000-0600
Gak! No!! Anyway, this could be useful, I mean, I know of some management applications that parse xml for other things, that could be adapted to use xml output from the manager interface.
By: Mark Spencer (markster) 2005-03-01 23:28:34.000-0600
No XML. Manager is very easy to parse today because it's variable: value. Unless you can show to me that XML represents things which *cannot* reasonably be represented with the existing syntax, i have absolutely no interest in adding it in any form or fashion.
By: Olle Johansson (oej) 2005-03-02 00:34:27.000-0600
Markster: let's discuss this next week, I think you are too stubborn and unflexible here. There's a lot of development tools out there that makes parsing XML much easier than parsing your own format.
By: Olle Johansson (oej) 2005-03-02 00:34:47.000-0600
--refusal by markster---
By: Olle Johansson (oej) 2005-03-02 12:20:15.000-0600
Will clean this up and remove XML within a week or so, have to travel a bit...
By: Olle Johansson (oej) 2005-03-09 01:17:54.000-0600
I can't figure out a good way to send multiple peers in a list. I do not think the status way of sending several channels is a good solution. Events should be unsolicited, requests should generate responses, not a list of events. Can we come up with a better syntax than that?
By: Mark Spencer (markster) 2005-03-09 01:37:34.000-0600
To do multiple peers in the list, look how "Status" manager event is handled. You first send an ack ("Channel status will follow"), followed by individual "events" or "cli_fd"'s for each peer, followed by "StatusComplete". For more information, see action_status in manager.c.
By: Olle Johansson (oej) 2005-03-09 23:15:01.000-0600
Yes, I've seen that (as mentioned above) and don't like the syntax, it's kind of weird. I'll see if I can come up with something similar...
By: Olle Johansson (oej) 2005-03-10 01:53:13.000-0600
Ok, take a look at this version. Based on the Status command...
By: Olle Johansson (oej) 2005-03-13 09:39:14.000-0600
markster: Please check when you have a moment... If this works, I'll start working on IAX2.
By: Olle Johansson (oej) 2005-03-17 10:40:41.000-0600
- Removed duplicate ack
- Removed Peerlist: in ACK
...according to Marks suggestions...
By: Mark Spencer (markster) 2005-03-20 01:59:16.000-0600
Added to CVS with minor mods. Thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:28:03.000-0600
r5215 | markster | 2008-01-15 15:28:02 -0600 (Tue, 15 Jan 2008) | 2 lines
Add SIPPeers and SIPShowPeer manager events (bug ASTERISK-3610 with mods)