Summary:ASTERISK-01948: [patch] Allow Manager API Command to return big amount of output data +++
Reporter:mochouinard (mochouinard)Labels:
Date Opened:2004-07-04 15:04:34Date Closed:2004-09-25 02:26:26
Versions:Frequency of
Environment:Attachments:( 0) cli.c.diff
( 1) cli.c.diff
Description:If you call Command via the Manager API and command return alot of output, the buffer get overloaded and skip information, so you dont get the complete information. BUT it could also affect alot of things, like the log files, console and different things. So I just made it to handle errors on write.


I had put that patch in 1972 because mustdie asked for that thing to work correctly, so it why he asked for adding a new database command in the api command.  BUT it AINT the same as bug 1110 OR 1967. This patch doesn't fit that problem, but fix the manager and probably alot of others text output sections. I'll continue to update the patch here
Comments:By: mochouinard (mochouinard) 2004-07-04 15:43:48

I didn't saw the bug 1906 that refer to the problem.. but NO it not a dupe of 1110, 1110 look like to handle thing differently than the manager API since this patch didn't fix it

By: James Golovich (jamesgolovich) 2004-07-04 15:44:03

We need to fix the real problem, not fix it with some hacks

By: mochouinard (mochouinard) 2004-07-04 15:53:33

what hack ? Can you elaborate ?

By: mochouinard (mochouinard) 2004-07-04 19:34:50

kram showed me a nice function in the manager interface  :
So I updated the patch to use it.

By: mustdie (mustdie) 2004-07-05 12:48:47

Excellent job. Please merge it into CVS.

By: twisted (twisted) 2004-07-08 08:44:51

again, this is not a BLOCK issue, since it does not put * in a hanging state.  This is a patch, to tweak the output buffer to display the full list of information.  Updated severity accordingly

By: mustdie (mustdie) 2004-07-08 11:41:07

It is a BLOCK issue for all people who uses Manager API and using at least 10Kb of CLI output (Database or Dialplan output for example). We can't rely on broken  output, so it is very important to merge it into CVS ASAP.

By: Malcolm Davenport (mdavenport) 2004-07-08 15:56:09

Fixed in CVS.  Thank you.