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 adjust the verbiage of Flink 2.x onwards and 
instead say:
   
   "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