rionmonster commented on code in PR #27548:
URL: https://github.com/apache/flink/pull/27548#discussion_r2776115284


##########
docs/content/docs/deployment/security/security-ssl.md:
##########
@@ -94,7 +94,7 @@ SSL can be enabled separately for *internal* and *external* 
connectivity:
   - **security.ssl.internal.enabled**: Enable SSL for all *internal* 
connections.
   - **security.ssl.rest.enabled**: Enable SSL for *REST / external* 
connections.
 
-*Note: For backwards compatibility, the **security.ssl.enabled** option still 
exists and enables SSL for both internal and REST endpoints.*
+*Note: **security.ssl.enabled** option has been removed and is not supported 
for backward compatibility from Flink 2.x onwards*

Review Comment:
   Minor nit: We may want to tweak the wording around "Flink 2.x onwards" and 
instead say something like:
   
   > Note: The **security.ssl.enabled** option has been removed and is no 
longer supported for backward compatibility as of Flink 2.x.



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