davsclaus commented on code in PR #8774:
URL: https://github.com/apache/camel/pull/8774#discussion_r1032016103


##########
components/camel-splunk-hec/src/main/java/org/apache/camel/component/splunkhec/SplunkHECConfiguration.java:
##########
@@ -99,6 +101,20 @@ public void setHost(String host) {
         this.host = host;
     }
 
+    /**
+     * Splunk endpoint
+     *   Defaults to /services/collector/event
+     *   To write RAW data like JSON use /services/collector/raw
+     *   To extract timestamps in Splunk>8.0 
/services/collector/event?auto_extract_timestamp=true
+     */
+    public void setEndpoint(String endpoint) {

Review Comment:
   Are there other endpoints? And endpoint can maybe confuse as Camel also has 
a concept of endpoint. So maybe we can come up with a different name?
   
   For the last with the query parameter then that will not work as Camel think 
its another parameter.
   
   So maybe come up
   
   1) a new name, with 3 enum values: default, raw, timestamp
   2) And then you can set the url based on the endpoint inside the camel 
component



-- 
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: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to