slachiewicz commented on issue #1990:
URL:
https://github.com/apache/maven-resolver/issues/1990#issuecomment-5499273387
### Analysis & Triage
- **Problem Identified**: `ChainedRepositoryListener` and
`ChainedTransferListener` have an empty default `handleError()` implementation,
causing any `RuntimeException` (such as `NullPointerException` or
`IllegalArgumentException`) thrown by custom listeners to be swallowed silently
with no logging or notification.
- **Affected Versions**: Both **1.9.x** and **2.0.x / master**.
- **Plan to Fix**:
1. Add an SLF4J logger to `ChainedRepositoryListener` and
`ChainedTransferListener` to log listener exceptions at `WARN` level by default.
2. Add unit tests in `ChainedRepositoryListenerTest` and
`ChainedTransferListenerTest` verifying that exceptions in one listener do not
prevent subsequent listeners in the multicast chain from receiving events.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
