snuyanzin commented on code in PR #27407:
URL: https://github.com/apache/flink/pull/27407#discussion_r2688157106


##########
flink-core/src/main/java/org/apache/flink/configuration/SecurityOptions.java:
##########
@@ -519,7 +519,7 @@ public static Configuration forProvider(Configuration 
configuration, String prov
     public static final ConfigOption<Boolean> SSL_VERIFY_HOSTNAME =
             key("security.ssl.verify-hostname")
                     .booleanType()
-                    .defaultValue(true)
+                    .defaultValue(false)

Review Comment:
   Changing the default value is very questionable step and shouldn't be done 
without justification and discussion in Mailing List imho



-- 
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]

Reply via email to