Summary: | ASTERISK-17664: [patch] FullyBooted event received even when events are disabled | ||
Reporter: | Ben Klang (bklang) | Labels: | |
Date Opened: | 2011-04-09 15:35:59 | Date Closed: | 2011-05-03 13:27:28 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Core/ManagerInterface |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) fullybooted-filter.patch ( 1) issue19089-1.8-r316204.patch | |
Description: | [Sat Apr 09 16:09:58]$ telnet localhost 5038 Trying ::1... telnet: connect to address ::1: Connection refused Trying fe80::1%lo0... telnet: connect to address fe80::1%lo0: Connection refused Trying 127.0.0.1... Connected to localhost (127.0.0.1). Escape character is '^]'. Asterisk Call Manager/1.1 Action: Login Username: adhearsion Secret: <redacted> Events: off Response: Success Message: Authentication accepted Event: FullyBooted Privilege: system,all Status: Fully Booted | ||
Comments: | By: Sean Bright (seanbright) 2011-05-03 12:49:04 Uploaded another patch - this one will obey what is passed to the Events header. If you pass: Events: on or Events: all or Events: system You will see the FullyBooted event, otherwise you won't. This work? By: Ben Klang (bklang) 2011-05-03 12:53:35 @seanbright, Yes, your patch is definitely better, thanks! By: Russell Bryant (russell) 2011-05-03 13:08:52 patch looks good to me, seanbright By: Digium Subversion (svnbot) 2011-05-03 13:27:17 Repository: asterisk Revision: 316206 U branches/1.8/main/manager.c ------------------------------------------------------------------------ r316206 | seanbright | 2011-05-03 13:27:16 -0500 (Tue, 03 May 2011) | 8 lines If we aren't interested in events, don't generate the FullyBooted event on AMI login. (closes issue ASTERISK-17664) Reported by: bklang Patches: issue19089-1.8-r316204.patch uploaded by seanbright (license 71) Tested by: seanbright ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=316206 By: Digium Subversion (svnbot) 2011-05-03 13:27:27 Repository: asterisk Revision: 316213 _U trunk/ U trunk/main/manager.c ------------------------------------------------------------------------ r316213 | seanbright | 2011-05-03 13:27:27 -0500 (Tue, 03 May 2011) | 15 lines Merged revisions 316206 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.8 ........ r316206 | seanbright | 2011-05-03 14:17:36 -0400 (Tue, 03 May 2011) | 8 lines If we aren't interested in events, don't generate the FullyBooted event on AMI login. (closes issue ASTERISK-17664) Reported by: bklang Patches: issue19089-1.8-r316204.patch uploaded by seanbright (license 71) Tested by: seanbright ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=316213 |