StarRocks / starrocks

The world's fastest open query engine for sub-second analytics both on and off the data lakehouse. With the flexibility to support nearly any scenario, StarRocks provides best-in-class performance for multi-dimensional analytics, real-time analytics, and ad-hoc queries. A Linux Foundation project.
https://starrocks.io
Apache License 2.0
9.17k stars 1.82k forks source link

[BugFix] enable http/1.1 when proxying http request for FE #53010

Closed kevincai closed 5 days ago

kevincai commented 6 days ago

Why I'm doing:

What I'm doing:

Refs #52998

What type of PR is this:

Does this PR entail a change in behavior?

If yes, please specify the type of change:

Checklist:

Bugfix cherry-pick branch check:

github-actions[bot] commented 6 days ago

[FE Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] commented 6 days ago

[Java-Extensions Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] commented 6 days ago

[BE Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

banmoy commented 6 days ago

Is this issue from the real use case? I haven't met this exception before.

kevincai commented 6 days ago

Is this issue from the real use case? I haven't met this exception before.

Introduced in line https://github.com/StarRocks/starrocks/pull/52582/files#diff-4fc2b2317e41e62c2dbbde01b47588b8b582fa39e29a74f2eeddc2b8b0533666R99

It enforces a strict check of Expect header with http version in context.

github-actions[bot] commented 5 days ago

@Mergifyio backport branch-3.3

github-actions[bot] commented 5 days ago

@Mergifyio backport branch-3.2

github-actions[bot] commented 5 days ago

@Mergifyio backport branch-3.1

mergify[bot] commented 5 days ago

backport branch-3.3

✅ Backports have been created

* [#53024 [BugFix] enable http/1.1 when proxying http request for FE (backport #53010)](https://github.com/StarRocks/starrocks/pull/53024) has been created for branch `branch-3.3`
mergify[bot] commented 5 days ago

backport branch-3.2

✅ Backports have been created

* [#53025 [BugFix] enable http/1.1 when proxying http request for FE (backport #53010)](https://github.com/StarRocks/starrocks/pull/53025) has been created for branch `branch-3.2`
mergify[bot] commented 5 days ago

backport branch-3.1

✅ Backports have been created

* [#53026 [BugFix] enable http/1.1 when proxying http request for FE (backport #53010)](https://github.com/StarRocks/starrocks/pull/53026) has been created for branch `branch-3.1`