Croway commented on code in PR #20767:
URL: https://github.com/apache/camel/pull/20767#discussion_r2711976256


##########
dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/action/CamelSendAction.java:
##########
@@ -115,10 +125,23 @@ public class CamelSendAction extends ActionBaseCommand {
     @CommandLine.Option(names = { "--logging-color" }, defaultValue = "true", 
description = "Use colored logging")
     boolean loggingColor = true;
 
+    @CommandLine.Option(names = { "--infra" },
+                        description = "Send to infrastructure service (e.g., 
nats, kafka)")
+    String infra;
+
     private volatile long pid;
 
     private MessageTableHelper tableHelper;
 
+    private final CamelCatalog catalog = new DefaultCamelCatalog();
+
+    // Common Camel framework properties that should not be auto-set from 
infra connection details.
+    // These are component lifecycle/behavior properties, not infrastructure 
connection properties.
+    private static final Set<String> EXCLUDED_COMPONENT_OPTIONS = Set.of(

Review Comment:
   Is it really needed? are there services that uses these properties?



-- 
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]

Reply via email to