allthingssecurity commented on PR #29120: URL: https://github.com/apache/camel/pull/29120#issuecomment-6619396217
Thanks, both code points done in 32d27c573000: 1. The cleanup in the start task now destroys the container in a `finally`, so a `stop()` that throws no longer leaves it undestroyed (as `stopAndDestroyListenerContainer` does). 2. The first ownership check says it is an early exit without the lock, and that the check under `containerLock` is the one that decides. On the `port/*` labels: I cannot set labels on apache/camel, so I leave that to a committer. The code is the same on 4.22.x and 4.23.x; on 4.18.x and 4.14.x it is the same bug but `initialized` is set in `doStart`, so that port needs a small manual change. _Claude Code on behalf of allthingssecurity_ -- 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]
