Summary:	ASTERISK-14808: [patch] SIPshowregistry manager action obmits ActionID from RegistryEntry events
Reporter:	Nic Bellamy (nic_bellamy)	Labels:
Date Opened:	2009-09-09 18:31:40	Date Closed:	2009-09-10 07:17:38
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Channels/chan_sip/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) manager_SIPshowregistry_actionid.patch
Description:	When performing a SIPshowregistry manager API action, the "Response: Success" and the "Event: RegistrationsComplete" responses both include the ActionID, but the "Event: RegistryEntry" events do not. The attached patch fixes the issue. Example of the problem: Action: SIPshowregistry ActionID: 1234 Response: Success ActionID: 1234 Eventlist: start Message: Registrations will follow Event: RegistryEntry Host: turbo Port: 5060 Username: 604 Refresh: 105 State: Registered RegistrationTime: 1252537136 Event: RegistrationsComplete EventList: Complete ListItems: 1 ActionID: 1234 ****** ADDITIONAL INFORMATION ****** Patch is against SVN trunk, but other 1.6.x branches have the same issue (originally found in 1.6.1.6)
Comments:	By: Olle Johansson (oej) 2009-09-10 07:06:05 You are very right. THanks for your contribution! By: Digium Subversion (svnbot) 2009-09-10 07:08:10 Repository: asterisk Revision: 217593 U   trunk/channels/chan_sip.c ------------------------------------------------------------------------ r217593 | oej | 2009-09-10 07:08:09 -0500 (Thu, 10 Sep 2009) | 8 lines Include ActionID in all events that are responsed to AMI Action SIPShowRegistry (closes issue ASTERISK-14808) Reported by: nic_bellamy Patches:      manager_SIPshowregistry_actionid.patch uploaded by nic bellamy (license 299) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=217593 By: Digium Subversion (svnbot) 2009-09-10 07:10:39 Repository: asterisk Revision: 217594 _U  branches/1.6.2/ U   branches/1.6.2/channels/chan_sip.c ------------------------------------------------------------------------ r217594 | oej | 2009-09-10 07:10:39 -0500 (Thu, 10 Sep 2009) | 16 lines Merged revisions 217593 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r217593 | oej | 2009-09-10 14:06:55 +0200 (Tor, 10 Sep 2009) | 8 lines Include ActionID in all events that are responsed to AMI Action SIPShowRegistry (closes issue ASTERISK-14808) Reported by: nic_bellamy Patches:      manager_SIPshowregistry_actionid.patch uploaded by nic bellamy (license 299) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=217594 By: Digium Subversion (svnbot) 2009-09-10 07:12:21 Repository: asterisk Revision: 217595 _U  branches/1.6.1/ U   branches/1.6.1/channels/chan_sip.c ------------------------------------------------------------------------ r217595 | oej | 2009-09-10 07:12:21 -0500 (Thu, 10 Sep 2009) | 16 lines Merged revisions 217593 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r217593 | oej | 2009-09-10 14:06:55 +0200 (Tor, 10 Sep 2009) | 8 lines Include ActionID in all events that are responsed to AMI Action SIPShowRegistry (closes issue ASTERISK-14808) Reported by: nic_bellamy Patches:      manager_SIPshowregistry_actionid.patch uploaded by nic bellamy (license 299) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=217595 By: Digium Subversion (svnbot) 2009-09-10 07:17:38 Repository: asterisk Revision: 217596 _U  branches/1.6.0/ U   branches/1.6.0/channels/chan_sip.c ------------------------------------------------------------------------ r217596 | oej | 2009-09-10 07:17:38 -0500 (Thu, 10 Sep 2009) | 16 lines Merged revisions 217593 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r217593 | oej | 2009-09-10 14:06:55 +0200 (Tor, 10 Sep 2009) | 8 lines Include ActionID in all events that are responsed to AMI Action SIPShowRegistry (closes issue ASTERISK-14808) Reported by: nic_bellamy Patches:      manager_SIPshowregistry_actionid.patch uploaded by nic bellamy (license 299) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=217596