elkhand commented on PR #23406:
URL: https://github.com/apache/flink/pull/23406#issuecomment-1724945763

   > Not sure I understand the feature from user perspective🤔 The new URL is 
clear, the existing booleans for SSL are clear. What I would like to ask is 
what happens when I set `SecurityOptions.SSL_REST_ENABLED = false` and 
`URL=https://...` and the other way around? I mean which part of the code is 
configured by the new path var and which part by the old boolean? I'm asking it 
because we're setting `SecurityOptions.SSL_REST_ENABLED` dynamically...
   
   Updated code.
   Now we set SSL_REST_ENABLED if URL starts with https only:
   
https://github.com/apache/flink/pull/23406/files#diff-2ebb5c60764d835a040c0e36aff8f42ae9a0239cb1186afd3d220b6e18d3dcb7R69
 


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to