elharo opened a new pull request, #2045: URL: https://github.com/apache/maven-resolver/pull/2045
Fixes #1994 `IpcNamedLock.tryUnlock()` was silently discarding all exceptions from `client.unlock()` during timeout cleanup, masking real IO/connectivity issues. **Fix:** log the failure at WARN level instead of swallowing it. The exception is still not propagated, preserving the best-effort cleanup semantics that must not crash the calling thread. **Test:** added `IpcNamedLockTest.unlockFailureDuringTimeoutCleanupIsLogged`, which mocks `IpcClient` so that `lock()` times out and `unlock()` throws, then asserts the unlock failure is logged. The test fails against the old code (no log output) and passes with the fix. -- 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]
