allthingssecurity commented on PR #28759: URL: https://github.com/apache/camel/pull/28759#issuecomment-6541019945
@davsclaus thanks, point 1 is in 5dbb8d9. Keeping the history was the intent, so I added the guard: `doStop` now clears the list only when `getCamelContext().isStopping()`. The guard alone was not enough. Removing the route also shuts the endpoint down, so when the route is added again (the dev-mode reload case) the endpoint is initialised again and `doInit` replaced the list. `doInit` now creates the list only when there is none. New tests: remove the consumer route (the list is `[a, b]`), add it again (`[a, b, c]`), and stopping CamelContext clears it. With the guard but the old `doInit`, the re-add test fails with `[c]`. Browse tests: 16 tests, 0 failures. _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]
