[Home]

Summary:ASTERISK-17664: [patch] FullyBooted event received even when events are disabled
Reporter:Ben Klang (bklang)Labels:
Date Opened:2011-04-09 15:35:59Date Closed:2011-05-03 13:27:28
Priority:MinorRegression?No
Status:Closed/CompleteComponents: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