allthingssecurity commented on PR #28750: URL: https://github.com/apache/camel/pull/28750#issuecomment-6540981743
@davsclaus thanks, point 1 is in f243730: the step callback in `RoutingSlip.processExchange` sets `CamelSlipEndpoint` back to the step's endpoint. Two new tests use an outer dynamic router that records the property. With a nested routing slip it used to see `mock://inner` and now sees `direct://sub`. With an empty nested slip it saw no endpoint at all, because a slip removes the property when it starts, so the router took its first step again and again; that test bounds it to five steps. Both fail without the line and pass with it. `*RoutingSlip*Test` and `*DynamicRouter*Test`: 91 tests, 0 failures. I left point 3 as it is. _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]
