search

enygma / expose

An Intrusion Detection System library loosely based on PHP IDS
MIT License
266 stars 55 forks source link

Restoring Original Notification Functionality #36

Closed charles-martin closed 9 years ago

charles-martin commented 9 years ago

Notification Functionality was broken/lost with the conversion to recursive filters checking. $filterMatches was updated on the recursion, but not on the process filters. As a result, all launches sent an empty array to the sendNotifications method. These two changes A) Restore the documented notifications system and B) Update the testing to properly ensure that a filter is sent to the notification system.

Restoring Original Filter Match Functionality To Fix Notifications Sending Updating Test To Verify Notification Called With Filters

enygma commented 9 years ago

Nice catch - thanks for the update! I'll make a new release with it here shortly.