Copilot commented on code in PR #26628:
URL: https://github.com/apache/camel/pull/26628#discussion_r4057127222
##########
components/camel-mail/src/main/java/org/apache/camel/component/mail/MailConfiguration.java:
##########
@@ -114,13 +114,13 @@ public class MailConfiguration implements Cloneable {
private boolean ignoreUnsupportedCharset;
@UriParam(label = "producer,advanced,security", security = "insecure:ssl")
private boolean useJavaMailSessionPropertiesFromHeaders;
- @UriParam(label = "producer")
+ @UriParam(defaultValue = "true", label = "producer")
Review Comment:
GitHub Copilot on behalf of Brijesh Thakkar: `defaultValue` only affects
generated metadata; it does not initialize the primitive fields. Since these
four fields still default to `false` and no constructor sets them, an endpoint
that omits these options continues to ignore message headers even though the
metadata and documentation now claim the default is `true`. Initialize each
field to `true` while retaining the explicit `false` override behavior.
--
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]
