CAMEL-24238: camel-aws2-ses - send RawMessage body content instead of the SdkBytes descriptor#25017
CAMEL-24238: camel-aws2-ses - send RawMessage body content instead of the SdkBytes descriptor#25017oscerd wants to merge 1 commit into
Conversation
… the SdkBytes descriptor
When the exchange body is an SES RawMessage, Ses2Producer built the outgoing
Content from the body converted to String, which for a RawMessage falls back
to Object.toString() on the SDK type. SdkBytes.toString() renders a debug
descriptor ("SdkBytes(bytes=0x46726f6d...)"), so the recipient received that
descriptor instead of the MIME payload.
Read the payload from RawMessage.data().asUtf8String() when the body is a
RawMessage, and cover it with a test asserting the sent text equals the
original MIME content.
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
(cherry picked from commit b100364)
gnodet
left a comment
gnodet
left a comment
There was a problem hiding this comment.
Clean backport of #25015 to camel-4.14.x. Identical fix (SdkBytes.toString() → asUtf8String()) and test. Cherry-pick applies cleanly.
Reviewed with Claude Code on behalf of gnodet. This review was generated by an AI agent and may contain inaccuracies; please verify all suggestions before applying.
gnodet
left a comment
gnodet
left a comment
There was a problem hiding this comment.
LGTM — clean fix for the SES RawMessage handling. The change correctly wraps the raw message content so that AWS SES processes it properly. Code style follows project conventions.
This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.
Claude Code on behalf of @gnodet
gnodet
left a comment
gnodet
left a comment
There was a problem hiding this comment.
Clean backport of PR #25015 to camel-4.14.x. The fix correctly replaces SdkBytes.toString() (which returns a debug descriptor) with SdkBytes.asUtf8String() (which returns the actual payload). The change is identical to the main-branch fix modulo expected branch differences (JUnit 5 vs 6). No scope creep, no unrelated changes, and CI passes.
This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.
Claude Code on behalf of Guillaume Nodet
3 participants
Backport of #25015 to
camel-4.14.x.Ses2Producer.createMessage()converted aRawMessagebody withraw.data().toString(), which yields theSdkBytesdebug descriptor(
SdkBytes(bytes=0x...)) rather than the MIME payload — so that descriptor wassent as the e-mail body. Fixed by reading
raw.data().asUtf8String().The bug is present on this branch verbatim. Cherry-picked cleanly;
SesComponentTestpasses (5/5) on this branch.Upgrade-guide note is not applicable — this is a plain bug fix with no
user-visible configuration change.
Claude Code on behalf of oscerd