Backport of #91437: explicit POCO request for REST-polaris catalog#1224
Backport of #91437: explicit POCO request for REST-polaris catalog#1224zvonand merged 1 commit intoantalya-25.8from
Conversation
explicit POCO request for REST-polaris catalog
QA VerificationThis change is difficult to validate end-to-end with a real Polaris setup, as Polaris is not trivial to deploy locally and there is no existing automated coverage for this specific OAuth behavior. To validate the fix, a local mock OAuth + REST catalog server was implemented to strictly validate Behavior comparisonBefore (current Antalya Docker release): After (this backport applied): This confirms that the OAuth request parameters are now properly URL-encoded and that ClickHouse successfully retrieves the access token and proceeds with authenticated REST catalog calls. CI resultsThere is no dedicated automated coverage for this specific OAuth behavior, so CI results were reviewed for regressions. Upstream CI for the original PR was not fully green. The backport CI also reported failures; however, they are consistent with upstream and previously known issues on the branch (e.g. Given the limited scope of the change (OAuth request formatting), this backport is considered safe and does not introduce regressions. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):
Backport of ClickHouse#91437
explicit POCO request for REST-polaris catalog
CI/CD Options
Exclude tests:
Regression jobs to run: