jamesnetherton commented on PR #29559: URL: https://github.com/apache/camel/pull/29559#issuecomment-6768100159
Thanks Claus. Addressed in b5b296f6bdeb: - **Status code in the jetty test.** Without an error handler the response is a 500, not a 400. `RestBindingAdvice` sets the 400 header, but the exception stays on the exchange, so the HTTP consumer answers 500. Only `clientRequestValidation` clears it. The test now does what `RestUndertowHttpPojoTypeTest` does: it answers the failure with 400 and its message, and checks that the message names the unknown property. An unrelated failure no longer passes it. - **`RestUndertowHttpPojoTypeTest`.** Dropped the explicit `FAIL_ON_UNKNOWN_PROPERTIES` and the stale comment. Its PUT cases send a body with only unknown properties and still get the 400, so the test now checks the new default on undertow. - **Package match.** Added a comment on `JACKSON3_PACKAGE`. - **Only `FAIL_ON_UNKNOWN_PROPERTIES`.** Yes, that was deliberate. It is the one changed default that relaxes validation of what external callers send. The others change the format of responses, and `FAIL_ON_NULL_FOR_PRIMITIVES` makes requests stricter, not looser. The upgrade guide table covers all of them. _Claude Code on behalf of jamesnetherton_ -- 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]
