fix: Notifications not being delivered to APKAM Monitors - Githubissues

atsign-foundation / at_server

The software implementation of Atsign's core technology

https://docs.atsign.com

BSD 3-Clause "New" or "Revised" License

39 stars 13 forks source link

fix: Notifications not being delivered to APKAM Monitors #1937

Closed gkc closed 2 months ago

gkc commented 2 months ago

- What I did

Ensured that properly-formed notifications are delivered to APKAM monitors

- How I did it

MonitorVerbHandler now uses the standard AbstractVerbHandler.isAuthorized check instead of its own custom code which was not working correctly for properly-formed notifications
Modified relevant monitor-plus-apkam unit tests so they create properly-formed notification objects for testing

- How to verify it Tests pass