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

[WFCORE-37] Don't throw an exception if a handler is added via the ad… #3435

Merged
merged 1 commit into from Aug 10, 2018

Conversation

jamezp
Copy link
Member

@jamezp jamezp commented Aug 10, 2018

…d-handler operation. Also fix the model validation to keep the handler names unique when associated with a logger or async-handler.

https://issues.jboss.org/browse/WFCORE-37

If adding a handler and then adding it to a logger or async-handler in a composite operation via the add-handler operation a failure would occur because the add operation already adds the handler as it's found in the model already. This just ignores the error and logs a trace message instead. The add-handler operation shouldn't fail if the handler is already assigned.

…d-handler operation. Also fix the model validation to keep the handler names unique when associated with a logger or async-handler.
@jmesnil jmesnil added ready-for-merge This PR is ready to be merged and fulfills all requirements 6.x labels Aug 10, 2018
@jmesnil jmesnil merged commit c73a491 into wildfly:master Aug 10, 2018
@jamezp jamezp deleted the WFCORE-37 branch March 13, 2019 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
2 participants