Skip to content

Add the ContentLengthHasBeenSet() method#35

Merged
alexey-milovidov merged 1 commit intomasterfrom
yarik/add-content-length-has-been-set
Mar 19, 2026
Merged

Add the ContentLengthHasBeenSet() method#35
alexey-milovidov merged 1 commit intomasterfrom
yarik/add-content-length-has-been-set

Conversation

@yariks5s
Copy link
Member

Issue #, if available:
ClickHouse/ClickHouse#99971

Description of changes:
This change is needed for the PR: . That PR fixes a case where ClickHouse can receive a file that does not have a Content-Length header in the HEAD request response, and therefore these files are skipped because their size was zero (default value). Required for ClickHouse/ClickHouse#99971

Check all that applies:

  • Did a review by yourself.
  • Added proper tests to cover this PR. (If tests are not applicable, explain.)
  • Checked if this PR is a breaking (APIs have been changed) change.
  • Checked if this PR will not introduce cross-platform inconsistent behavior.
  • Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

  • Linux
  • Windows
  • Android
  • MacOS
  • IOS
  • Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@alexey-milovidov alexey-milovidov self-assigned this Mar 19, 2026
@alexey-milovidov alexey-milovidov merged commit 04a2032 into master Mar 19, 2026
0 of 3 checks passed
@yariks5s yariks5s deleted the yarik/add-content-length-has-been-set branch March 19, 2026 14:28
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.

2 participants