allthingssecurity commented on code in PR #27467:
URL: https://github.com/apache/camel/pull/27467#discussion_r4205625982
##########
core/camel-core-engine/src/main/java/org/apache/camel/impl/DefaultModel.java:
##########
@@ -645,6 +645,21 @@ protected String doAddRouteFromTemplate(
if (group != null) {
def.setGroup(group);
}
+ // a route from a template (or a Kamelet) must not replace an existing
route with the same id, unless that
+ // route was created from the same template (such as the route of a
Kamelet with a route id that is created
+ // again when its parent route is updated)
Review Comment:
Thanks, right: the guard also rejects a route that exists in the
CamelContext without a route definition in the model. The comment now says so
(c8e1a471250f).
_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]
