azagrebin opened a new pull request #6796: [FLINK-10075] Redirect non-ssl requests to https url if ssl is enabled URL: https://github.com/apache/flink/pull/6796 ## What is the purpose of the change This PR introduces a wrapper around SSL handler in rest server endpoint. The wrapper checks whether the request is encrypted and processes normally only encrypted requests. The non-encrypted requests are redirected to advertised host with https protocol. The path is added to the redirect url if the request is http and the path is available. ## Brief change log - add RedirectingSslHandler wrapper for SslHandler with redirecting in newNonSslHandler() - replace SslHandler in RestServerEndpoint with RedirectingSslHandler - add unit test checking the redirection RestServerEndpointITCase.testNonSslRedirectForEnabledSsl() ## Verifying this change unit tests, try access Flink UI with http url and ssl enabled in flink.conf ## Does this pull request potentially affect one of the following parts: - Dependencies (does it add or upgrade a dependency): (no) - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: (no) - The serializers: (no) - The runtime per-record code paths (performance sensitive): (no) - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no) - The S3 file system connector: (no) ## Documentation - Does this pull request introduce a new feature? (no) - If yes, how is the feature documented? (not applicable)
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
