-
Notifications
You must be signed in to change notification settings - Fork 6.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add function getHttpHeader #54813
Add function getHttpHeader #54813
Conversation
756bf91
to
5db521e
Compare
This is an automated comment for commit 3bcd0b5 with description of existing statuses. It's updated for the latest CI running ✅ Click here to open a full report in a separate page Successful checks
|
d7c51bd
to
e22b5d6
Compare
cfaccae
to
5a8c40d
Compare
5a8c40d
to
0d256a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
create table headers(value String) engine = Memory;
insert into headers values ('X-Clickhouse-User')
and then doing
echo "SELECT getHttpHeader(value) from (select * from headers)" | curl -s -H 'X-ClickHouse-User: default' -H 'X-ClickHouse-Key: ' 'http://localhost:8123/' -d @-
doesn't work
caa0222
to
eab7b34
Compare
@antonio2368 Hello, could you help me to fix this unit test? I don't know what happend. |
@lingtaolf just in case update with the latest master? |
|
@antonio2368 All tests have passed, can this pr be merged? |
#56109 |
#54665
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Added new function
getHttpHeader
to get HTTP request header value used for a request to ClickHouse server. Return empty string if the request is not done over HTTP protocol or there is no such header.Documentation entry for user-facing changes