Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check for missing event handler #8

Open
CumpsD opened this issue Aug 12, 2019 · 0 comments
Open

Check for missing event handler #8

CumpsD opened this issue Aug 12, 2019 · 0 comments

Comments

@CumpsD
Copy link
Contributor

CumpsD commented Aug 12, 2019

Our logging sometimes shows a warning of this kind:

Could not resolve a handler for AddressPositionWasRemoved.

This is caused by events coming in, which are missing an event handler. This is most likely to happen in the syndication handlers.

This warning is noise since it requires no action. We have simply decided for event X we are not interested in it, which is a valid use case.

This has to be made more explicit by actually handling that event and doing nothing with it, instead of generating a warning.

Please review this registry for occurrences like this and fix them.

@CumpsD CumpsD assigned ArneD and unassigned ArneD Aug 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants