[PR] [backport camel-4.18.x] CAMEL-23627: fixed response error check in SlackProducer [camel]

["davsclaus (via GitHub)" <gi...@apache.org>]

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

   ## Backport of #23567
   
   Cherry-pick of #23567 onto `camel-4.18.x`.
   
   **Original PR:** #23567 - CAMEL-23627: fixed response error check in SlackProducer
   **Original author:** @Xadhoodin
   **Target branch:** `camel-4.18.x`
   
   ### Original description
   
   Move the Slack API error response check inside the try block so the exception is set
   on the exchange before callback.done(true) fires. Previously the check happened after
   the callback, causing onException policies to miss the error. Also fix both methods
   to return true since SlackProducer is fully synchronous.


-- 
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] [backport camel-4.18.x] CAMEL-23627: fixed response error check in SlackProducer [camel]

["davsclaus (via GitHub)" <gi...@apache.org>]

davsclaus merged PR #23574:
URL: https://github.com/apache/camel/pull/23574


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