Skip to content

Conversation

andris-rauda-sumsub
Copy link

Summary

Try implementing sending statement parameters via multipart/form-data to mitigate issues with too long url.

PR is a work in progress.

Tried to implement passing query parameters as Content-Type: multipart/form-data, got some ugly mockup (because of a streaming support), that kind of does work, however, it turns out, that when calling it with many parameters from local integration test, it starts failing with a weird server error even sooner, than the original application/x-www-form-urlencoded variant:

com.clickhouse.client.api.ServerException: Code: 1000. DB::Exception: <Unreadable error message> (transport error: 500)

So, seems, for now it is unfeasible to pursue this.

Closes

Checklist

Delete items not relevant to your PR:

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@chernser chernser requested review from mzitnik and Copilot September 4, 2025 14:05
@chernser
Copy link
Contributor

chernser commented Sep 4, 2025

Good day, @andris-rauda-sumsub!
Thank you for starting the contribution!

This may be would be easier to implement after we have organize a separate transport layer with multiple http client support.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Query parameters can't be sent through POST body
3 participants