gnodet-bot commented on code in PR #27482:
URL: https://github.com/apache/camel/pull/27482#discussion_r4205380840
##########
components/camel-aws/camel-aws2-transcribe/src/main/java/org/apache/camel/component/aws2/transcribe/Transcribe2Configuration.java:
##########
@@ -45,7 +45,7 @@ public class Transcribe2Configuration implements Cloneable,
AwsCommonConfigurati
private boolean overrideEndpoint;
@UriParam
private String uriEndpointOverride;
- @UriParam(security = "insecure:ssl", defaultValue = "true")
+ @UriParam(security = "insecure:ssl", defaultValue = "false")
Review Comment:
ℹ️ **Note:** The `defaultValue` change from `"true"` to `"false"` in the
`@UriParam` annotation is a **metadata-only correction** — the field `private
boolean trustAllCertificates;` has no explicit initializer, so the JVM
primitive default was always `false` at runtime. The old annotation was lying.
No behavioral change, but worth clarifying in the commit message to avoid
confusion during backport triage.
--
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]
