Summary: | ASTERISK-11240: [patch] Wrong value in XML HTTP manager response | ||
Reporter: | Marcos Jose Setim (msetim) | Labels: | |
Date Opened: | 2008-01-15 07:14:28.000-0600 | Date Closed: | 2008-02-13 00:33:00.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Core/ManagerInterface |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) 1.4.17_With_Patch.txt ( 1) ami_agent_talkingto-1.4.diff ( 2) info.txt | |
Description: | Hi guys, I was working with HTTP manager using XML responses when I saw it: Running http://172.16.0.75/oper/mxml?action=agents I get a wrong value to field talking to. See attach. Note that in XML response the field talking to say: "400" and from CLI say "SIP/115-etc" | ||
Comments: | By: Caio Begotti (caio1982) 2008-01-15 07:16:50.000-0600 Talking to msetim on IRC I guess I have a fix for this (actually it looks like the same code from trunk, I tested the patch last night on a clean 1.4 checkout). And somebody could update this issue to set the category to chan_agent and [patch] in the title? By: Michiel van Baak (mvanbaak) 2008-01-15 07:57:06.000-0600 Updated on caio1982's request on IRC By: Jason Parker (jparker) 2008-01-15 10:04:42.000-0600 This is a duplicate of ASTERISK-9792. Changing this in 1.4 would be a change in behavior, and cannot be done. I would consider this to be a feature rather than a bug. Please see ASTERISK-9792 for a patch that you can use to get the behavior you want. Closing. By: Jason Parker (jparker) 2008-01-15 11:05:10.000-0600 After much discussion with caio1982, we've decided to reopen this, as the issue is not quite the same as I thought it was. The issue here is not, as ASTERISK-9792 reported, that the CLI and manager event don't match "SIP/bob-blah", it's that the callerid number on the manager event is incorrect. It *should* be showing the bridged channel's callerid, but instead it is showing the logged in agent's callerid. By: Marcos Jose Setim (msetim) 2008-01-15 11:14:27.000-0600 Yeah! It was what i want to say :-D Thank you caio1982 and Qwell. Sorry for my poor English ;) By: Marcos Jose Setim (msetim) 2008-01-17 13:49:32.000-0600 Hi, I have installed asterisk 1.4.17 with patch applied and it's working. HTTP Manager is showing the correct talkingto. Do you can see it at file attached. Scenario: Queue: 800 Agent: agent/600 SIP/101 make a call to queue 800 and is talking with agent/600. By: Digium Subversion (svnbot) 2008-02-13 00:24:31.000-0600 Repository: asterisk Revision: 103607 U branches/1.4/channels/chan_agent.c ------------------------------------------------------------------------ r103607 | tilghman | 2008-02-13 00:24:30 -0600 (Wed, 13 Feb 2008) | 7 lines We aren't talking to ourselves; we're talking to someone else. (closes issue ASTERISK-11240) Reported by: msetim Patches: ami_agent_talkingto-1.4.diff uploaded by caio1982 (license 22) Tested by: caio1982, msetim ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=103607 By: Digium Subversion (svnbot) 2008-02-13 00:32:14.000-0600 Repository: asterisk Revision: 103608 _U trunk/ ------------------------------------------------------------------------ r103608 | tilghman | 2008-02-13 00:32:12 -0600 (Wed, 13 Feb 2008) | 14 lines Blocked revisions 103607 via svnmerge ........ r103607 | tilghman | 2008-02-13 00:25:03 -0600 (Wed, 13 Feb 2008) | 7 lines We aren't talking to ourselves; we're talking to someone else. (closes issue ASTERISK-11240) Reported by: msetim Patches: ami_agent_talkingto-1.4.diff uploaded by caio1982 (license 22) Tested by: caio1982, msetim ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=103608 By: Tilghman Lesher (tilghman) 2008-02-13 00:33:00.000-0600 Blocked for trunk, as a variation of this patch is already there. |