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

   Child of CAMEL-24261 (completing CAMEL-23462 across the AWS2 producers).
   
   ## Problem
   
   With `pojoRequest=true`, `EventbridgeProducer.putRule` only acted when the 
body
   was a `PutRuleRequest`; any other body fell through the `instanceof` with no
   `else`, so no AWS call was made, no response was set, and the original body 
was
   returned with no error. (The other Eventbridge operations already build their
   request from headers and don't have a pojo branch.)
   
   ## Fix
   
   ```java
   } else {
       throw new IllegalArgumentException(
               "putRule operation requires PutRuleRequest in POJO mode");
   }
   ```
   
   ## Test
   
   `camel-aws2-eventbridge` has no producer route/mock harness, so this is 
covered
   by a new Mockito unit test (`EventbridgeProducerPojoRequestTest`) that mocks 
the
   endpoint + client, sends a wrong-typed body with `pojoRequest=true`, and 
asserts
   the `IllegalArgumentException`. Verified to fail (silent no-op) before the 
fix.
   Region-free — no localstack or real AWS.
   
   ## Docs / backport
   
   The shared 4.22 upgrade-guide entry was added with CAMEL-24263. Main only, 
matching
   the CAMEL-23462 precedent (behaviour change, shipped in a minor, not 
backported).
   
   ---
   _Claude Code on behalf of oscerd_


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