mateczagany commented on code in PR #27457:
URL: https://github.com/apache/flink/pull/27457#discussion_r2716079197


##########
flink-core/src/main/java/org/apache/flink/configuration/SecurityOptions.java:
##########
@@ -514,15 +523,6 @@ public static Configuration forProvider(Configuration 
configuration, String prov
                                                     "here"))
                                     .build());
 
-    /** Flag to enable/disable hostname verification for the ssl connections. 
*/
-    @Documentation.Section(Documentation.Sections.SECURITY_SSL)
-    public static final ConfigOption<Boolean> SSL_VERIFY_HOSTNAME =

Review Comment:
   If you prefer to have it marked as deprecated, we can do that, but overall 
this configuration should have no effect to stay consistent with previous Flink 
releases.
   
   We can note in the description that the user should use the new 
configuration option to have hostname verification for external connections.



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