gnodet opened a new pull request, #21921:
URL: https://github.com/apache/camel/pull/21921

   ## Summary
   
   Follow-up fixes for the Jackson 3 migration (#21800). These address issues 
found during code review.
   
   ### Fixes included:
   
   - **Salesforce `JsonUtils`**: Restore `WRITE_DATES_AS_TIMESTAMPS` and 
`ADJUST_DATES_TO_CONTEXT_TIME_ZONE` using Jackson 3's `DateTimeFeature` — 
without this, dates serialize as numeric timestamps
   - **ServiceNow `ServiceNowConfiguration`**: Restore custom date/time format 
serializers using the user's `dateFormat`/`timeFormat`/`dateTimeFormat` config 
properties, which became dead code after migration
   - **CBOR `CBORDataFormat`**: Fix registry lookup to find `ObjectMapper` 
instances (not `TokenStreamFactory`) so users with registered `CBORMapper` are 
still auto-discovered; consolidate repeated `rebuild().build()` into single call
   - **Micrometer `MicrometerModule`**: Remove dead `throws IOException` and 
try-catch from serializer methods (Jackson 3 uses unchecked exceptions)
   - **Salesforce exception handling**: Restore try-catch in 
`AbstractDTOBase.toString()`, `LoginConfigHelper`, `StreamingApiConsumer`, and 
`DefaultBulkApiV2Client` — Jackson 3's unchecked `JacksonException` was causing 
lost error handling
   
   ### Context
   
   See review comment: 
https://github.com/apache/camel/pull/21800#issuecomment-4037001940
   
   This PR should be merged after #21800.


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