davsclaus commented on PR #29810: URL: https://github.com/apache/camel/pull/29810#issuecomment-6848475937
Thanks for jumping on this. Your guess is right: since #29753 the pulsar `producerIn` body type `dependsOn` `allowJavaSerializedObject`, and as that option can also be set on the component, `BodyTypeResolver` treats the body as not known when the endpoint does not set it, so the finding is gone. #29807 (opened a bit earlier, CI running) makes the same test change as this PR, to get `main` green quickly. The real fix is #29808 (CAMEL-26916, stacked on #29807): an insecure opt-in (`security = "insecure:..."`, such as `allowJavaSerializedObject`) keeps its default in the validator unless the step sets it, since one turned on for the whole component is reported by the security policy at startup. With it, the pulsar finding comes back and the test expects 4 again. So I suggest closing this one in favour of #29807 + #29808. _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]
