Skip to content

Allow empty HTTP headers#24285

Merged
alexey-milovidov merged 2 commits intoClickHouse:masterfrom
abyss7:issue-23901
May 21, 2021
Merged

Allow empty HTTP headers#24285
alexey-milovidov merged 2 commits intoClickHouse:masterfrom
abyss7:issue-23901

Conversation

@abyss7
Copy link
Copy Markdown
Contributor

@abyss7 abyss7 commented May 19, 2021

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Allow empty HTTP headers. Fixes #23901

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label May 20, 2021
@alexey-milovidov alexey-milovidov self-assigned this May 21, 2021
@alexey-milovidov alexey-milovidov merged commit 5de0339 into ClickHouse:master May 21, 2021
@abyss7 abyss7 deleted the issue-23901 branch May 21, 2021 11:04
alexey-milovidov added a commit that referenced this pull request May 21, 2021
Backport #24285 to 21.6: Allow empty HTTP headers
alexey-milovidov added a commit that referenced this pull request May 21, 2021
Backport #24285 to 21.5: Allow empty HTTP headers
alexey-milovidov added a commit that referenced this pull request May 21, 2021
Backport #24285 to 21.4: Allow empty HTTP headers
nikitamikhaylov added a commit that referenced this pull request Jun 2, 2021
Backport #24285 to 21.3: Allow empty HTTP headers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP request with an empty header value results in bad request response

3 participants