[CAMEL-16086] camel-salesforce - CompositeApiProcessor requires too many fields in RAW mode

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.7.1
• Fix Version/s: 3.8.0
• Component/s: camel-salesforce
• Estimated Complexity: Unknown
• Assignee: Jeremy Ross
• Reporter: Jesse Sightler
• Created: 26/Jan/21 22:42
• Updated: 17/Mar/21 17:07
• Resolved: 06/Feb/21 18:32

Description

https://github.com/apache/camel/blob/ae2c79dcd64424acc82a29c9a46620809ea6414c/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/processor/CompositeApiProcessor.java#L251https://github.com/apache/camel/blob/master/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/processor/CompositeApiProcessor.java#L251

 

This requires the SOBJECT_EXT_ID_NAME, but this should not be required. SFDC Documentation: https://developer.salesforce.com/docs/atlas.en-us.api_rest.meta/api_rest/resources_composite_composite.htm

In our case, we are using platform events, and upsert ids would not make sense for us.

 

Also this line is requiring a COMPOSITE_METHOD: https://github.com/apache/camel/blob/ae2c79dcd64424acc82a29c9a46620809ea6414c/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/processor/CompositeApiProcessor.java#L252

 

This seems unnecessary, as it is already defaulted to POST in DefaultCompositeApiClient: https://github.com/apache/camel/blob/ae2c79dcd64424acc82a29c9a46620809ea6414c/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/client/DefaultCompositeApiClient.java#L106

 

 

Issue Links

• relates to: CAMEL-16368 camel-salesforce: Add support for Composite sObject Collections API
• links to: GitHub Pull Request #5033