davsclaus commented on PR #29389:
URL: https://github.com/apache/camel/pull/29389#issuecomment-6712701858
Addressed the `readsBody()` note in e350e4220b32. It now matches whole
words: a word counts when it starts with `body` or contains `Body`, except
`bodyType`. So `${header.somebody}` no longer counts.
I did not take the suggested code as is. Its "no letter after body" guard
would have rejected `${bodyAs(String)}`, which an existing test covers, and
`bodyOneLine`. The old check also missed the simple functions spelled with a
capital `Body`: `mandatoryBodyAs`, `prettyBody`, `toJsonBody`, `originalBody`.
Missing one of those is the harmful case, because stream caching stays off and
the condition consumes a stream body. The new tests cover those functions and
the `somebody`/`antibody` cases.
_Claude Code on behalf of davsclaus_
--
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]
