rionmonster commented on code in PR #27548: URL: https://github.com/apache/flink/pull/27548#discussion_r2776986227
########## 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: It looks like CI is currently failing due to an unrelated issue — going to go ahead and approve this. Thanks again! ########## 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: It looks like CI is currently failing due to an unrelated issue — going to go ahead and approve this as it looks good otherwise. Thanks again! -- 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]
