Aayush10016 opened a new pull request, #105:
URL: https://github.com/apache/maven-shared-io/pull/105
## Summary
- avoid dereferencing a null thread context class loader
- return the existing unresolved-location result and diagnostic instead of
throwing
- add a regression test that restores the original context class loader
after verification
## Root cause
`ClasspathResourceLocatorStrategy.resolve` called `getResource`
unconditionally on `Thread.currentThread().getContextClassLoader()`. The
context class loader is allowed to be null in some runtime environments, which
caused a `NullPointerException`.
## Testing
- `mvn -Dtest=ClasspathResourceLocatorStrategyTest test`
- 5 tests passed
- `mvn -Prun-its verify`
- 80 tests passed
- Checkstyle, Spotless, Apache RAT, packaging, and the integration-test
profile succeeded
Fixes #90
--
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]
