You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "shaipan (via GitHub)" <gi...@apache.org> on 2026/04/28 12:16:04 UTC
[PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
shaipan opened a new pull request, #22799:
URL: https://github.com/apache/camel/pull/22799
### Summary
Add support for the `EventUuid` field returned by the Salesforce
Streaming API in change events, platform events, and generic events.
### Changes
- Added `HEADER_SALESFORCE_EVENT_UUID` (`CamelSalesforceEventUuid`)
constant to `SalesforceConstants`
- Updated `StreamingApiConsumer` to extract and set the `EventUuid`
header for all three event processing methods:
- Change events
- Platform events
- Generic/topic events
### Motivation
Salesforce includes an `EventUuid` field in Streaming API event
payloads which uniquely identifies each event. Without this header,
consumers had no way to access this value from Camel message headers.
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "Croway (via GitHub)" <gi...@apache.org>.
Croway commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4342607398
@shaipan you can back port the commit to `camel-4.14.x` branch, so that you will have the fix in the next 4.14 release, that will be `4.14.8`
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4338099172
<!-- ci-tested-modules -->
:test_tube: **CI tested the following changed modules:**
- `catalog/camel-catalog`
- `components/camel-salesforce/camel-salesforce-component`
- `dsl/camel-endpointdsl`
<details><summary>All tested modules (12 modules)</summary>
- Camel :: Catalog :: Camel Catalog
- Camel :: Endpoint DSL
- Camel :: JBang :: MCP
- Camel :: JBang :: Plugin :: Route Parser
- Camel :: JBang :: Plugin :: TUI
- Camel :: JBang :: Plugin :: Validate
- Camel :: Launcher :: Container
- Camel :: Salesforce
- Camel :: Salesforce :: CodeGen
- Camel :: Salesforce :: Maven Plugin
- Camel :: YAML DSL :: Validator
- Camel :: YAML DSL :: Validator Maven Plugin
</details>
---
:gear: [View full build and test results](https://github.com/apache/camel/actions/runs/25063767089)
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "shaipan (via GitHub)" <gi...@apache.org>.
shaipan commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4340784167
Hi @davsclaus Thanks for the merge? How can i take this into version 4.14.0?
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "Croway (via GitHub)" <gi...@apache.org>.
Croway commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4335284586
There are uncommitted changes, please build these modules to generate the missing code:
```
Changes not staged for commit:
(use "git add <file>..." to update what will be committed)
(use "git restore <file>..." to discard changes in working directory)
modified: catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/components/salesforce.json
modified: components/camel-salesforce/camel-salesforce-component/src/generated/resources/META-INF/org/apache/camel/component/salesforce/salesforce.json
modified: components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/StreamingApiConsumer.java
modified: dsl/camel-endpointdsl/src/generated/java/org/apache/camel/builder/endpoint/dsl/SalesforceEndpointBuilderFactory.java
```
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "shaipan (via GitHub)" <gi...@apache.org>.
shaipan commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4337199259
Hi @davsclaus can you please run the workflow now? I see it has all the files in PR.
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "shaipan (via GitHub)" <gi...@apache.org>.
shaipan commented on code in PR #22799:
URL: https://github.com/apache/camel/pull/22799#discussion_r3154139682
##########
components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/SalesforceConstants.java:
##########
@@ -23,6 +23,8 @@ public final class SalesforceConstants {
// Streaming headers
@Metadata(label = "consumer", description = "The Streaming API replayId.", javaType = "Object")
public static final String HEADER_SALESFORCE_REPLAY_ID = "CamelSalesforceReplayId";
+ @Metadata(label = "consumer", description = "The Streaming API eventUuid.", javaType = "Object")
Review Comment:
Kept it Object in the same lines as replayId
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "shaipan (via GitHub)" <gi...@apache.org>.
shaipan commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4342696026
@Croway added https://github.com/apache/camel/pull/22824
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4335148366
:star2: Thank you for your contribution to the Apache Camel project! :star2:
:robot: CI automation will test this PR automatically.
:camel: Apache Camel Committers, please review the following items:
* First-time contributors **require MANUAL approval** for the GitHub Actions to run
* You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot although they are normally detected and executed by CI.
* You can label PRs using `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR.
* Build and test logs are available in the summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary.
:warning: Be careful when sharing logs. Review their contents before sharing them publicly.
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4337053078
you need to build the entire code base, run
mvn clean install -P fastinstall
in the root.
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on code in PR #22799:
URL: https://github.com/apache/camel/pull/22799#discussion_r3154607161
##########
components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/SalesforceConstants.java:
##########
@@ -23,6 +23,8 @@ public final class SalesforceConstants {
// Streaming headers
@Metadata(label = "consumer", description = "The Streaming API replayId.", javaType = "Object")
public static final String HEADER_SALESFORCE_REPLAY_ID = "CamelSalesforceReplayId";
+ @Metadata(label = "consumer", description = "The Streaming API eventUuid.", javaType = "Object")
Review Comment:
I think we should look at all these headers what the actual java type that salesforce use so we can mark it correctly, String, Boolean, Integer etc. But this can be in another PR
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus merged PR #22799:
URL: https://github.com/apache/camel/pull/22799
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "shaipan (via GitHub)" <gi...@apache.org>.
shaipan commented on PR #22799:
URL: https://github.com/apache/camel/pull/22799#issuecomment-4337037656
@davsclaus @Croway why the build is stilll failing and it says "There are uncommitted changes"
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] camel-salesforce: Add EventUuid header support in Streaming API consumer [camel]
Posted by "Croway (via GitHub)" <gi...@apache.org>.
Croway commented on code in PR #22799:
URL: https://github.com/apache/camel/pull/22799#discussion_r3154102033
##########
components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/SalesforceConstants.java:
##########
@@ -23,6 +23,8 @@ public final class SalesforceConstants {
// Streaming headers
@Metadata(label = "consumer", description = "The Streaming API replayId.", javaType = "Object")
public static final String HEADER_SALESFORCE_REPLAY_ID = "CamelSalesforceReplayId";
+ @Metadata(label = "consumer", description = "The Streaming API eventUuid.", javaType = "Object")
Review Comment:
the javaType can be String here
--
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-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org