Croway opened a new pull request, #23300:
URL: https://github.com/apache/camel/pull/23300
## Summary
_Claude Code on behalf of Federico Mariani_
- **LogProcessor**: guard against null message before calling `doLog()` —
prevents NPE after CAMEL-22640 added `Objects.requireNonNull` to `CamelLogger`.
When a Simple expression like `${body}` evaluates to null (e.g.,
`ignoreBody=true` on the S3 consumer), the log EIP crashed with
`NullPointerException: message`.
- **AWS2S3Consumer**: use `Optional.ofNullable` instead of `Optional.of` in
`includeS3Object()` to handle null `contentType` when `includeFolders` is
`false`. Latent bug — currently dormant because `includeFolders` defaults to
`true`.
- **KinesisResumeStrategyConfiguration**: skip `setResumeCache` when cache
is null to avoid passing null downstream.
## Test plan
- [x] `S3ConsumerIgnoreBodyIT` — was consistently timing out, now passes
- [x] `S3ConsumerIT` — no regression
- [x] `S3ConsumerIncludeBodyIT` — no regression
--
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]
