gnodet-bot commented on code in PR #27765:
URL: https://github.com/apache/camel/pull/27765#discussion_r4256200822
##########
components/camel-http-common/src/main/java/org/apache/camel/http/common/HttpCommonEndpoint.java:
##########
@@ -84,9 +84,9 @@ public abstract class HttpCommonEndpoint extends
DefaultEndpoint
+ " data from the request to Java and that can be
a potential security risk.",
security = "insecure:serialization")
boolean transferException;
- @UriParam(label = "consumer",
+ @UriParam(label = "consumer", defaultValue = "true",
description = "If enabled and an Exchange failed processing on
the consumer side the response's body won't contain the exception's stack
trace.")
- boolean muteException;
+ boolean muteException = true;
Review Comment:
The change is correct. Both the `@UriParam defaultValue` and the field
initializer now say `true`, matching what
`HttpCommonComponent.createEndpoint()` has always set via
`setMuteException(isMuteException())`. No behavioral change — the `false`
default was dead code overwritten before any endpoint was used.
--
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]
