You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Abhishek Rawat (Code Review)" <ge...@cloudera.org> on 2021/12/15 22:27:06 UTC

[Impala-ASF-CR] IMPALA-11035: Make x-forwarded-for http header case-insensitive

Abhishek Rawat has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/18048 )

Change subject: IMPALA-11035: Make x-forwarded-for http header case-insensitive
......................................................................

IMPALA-11035: Make x-forwarded-for http header case-insensitive

Updated THttpServer::parseHeader to treat 'X-Forwarded-For' header name
as case-insensitive.

Added fe test (LdapHS2Test) to use mixed case 'x-forwarded-for' header.

Change-Id: Id9c4070a4a2d5ad9decb186a9219957d8c26a7d7
---
M be/src/transport/THttpServer.cpp
M fe/src/test/java/org/apache/impala/customcluster/LdapHS2Test.java
2 files changed, 22 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/48/18048/5
-- 
To view, visit http://gerrit.cloudera.org:8080/18048
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id9c4070a4a2d5ad9decb186a9219957d8c26a7d7
Gerrit-Change-Number: 18048
Gerrit-PatchSet: 5
Gerrit-Owner: Abhishek Rawat <ar...@cloudera.com>
Gerrit-Reviewer: Abhishek Rawat <ar...@cloudera.com>
Gerrit-Reviewer: Bikramjeet Vig <bi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>