liujiwen-up opened a new pull request, #675: URL: https://github.com/apache/doris-flink-connector/pull/675
# Proposed changes Issue Number: close #xxx ## Problem Summary: Add optional HTTPS support for Doris HTTP REST requests in the Flink connector while keeping HTTP as the default behavior. This change introduces HTTPS-related options aligned with the Spark connector: - `doris.enable.https` - `doris.https.key-store-path` - `doris.https.key-store-type` - `doris.https.key-store-password` The HTTP protocol switch is applied to Stream Load, Batch Stream Load, Stream Load 2PC commit/abort, FE REST requests, backend discovery, schema change, and COPY-related HTTP requests. For Stream Load redirection, the connector keeps the current Spark-compatible behavior: when `auto-redirect` is enabled, redirects are still handled by Apache HttpClient. If FE currently returns an HTTP `Location`, the request follows that redirect instead of failing fast, which preserves compatibility with existing Doris FE behavior. Once FE supports HTTPS redirect locations, the same implementation can support full HTTPS flow. ## Checklist(Required) 1. Does it affect the original behavior: No 2. Has unit tests been added: Yes 3. Has document been added or modified: No Need 4. Does it need to update dependencies: No 5. Are there any changes that cannot be rolled back: No ## Further comments The implementation keeps HTTP as the default protocol to avoid breaking existing deployments. HTTPS is enabled only through explicit configuration. Verified with: - `mvn -Pflink1 -pl flink-doris-connector-base -Dtest=DorisUrlBuilderTest,DorisOptionsTest,TestRestService,TestHttpUtil,TestBackendUtil,TestDorisStreamLoad,TestDorisCommitter,TestDorisBatchStreamLoad,SchemaManagerTest,TestDorisCopyCommitter test` - `mvn -Pflink1 -pl flink-doris-connector-flink1 -am -DskipTests package` - `mvn -Pflink2 -pl flink-doris-connector-flink2 -am -DskipTests package` - `git diff --check` -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
