Hello Attila Jeges, Tim Armstrong, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14149

to look at the new patch set (#2).

Change subject: Fix THttpServer to not call the cookie function with an empty 
cookie
......................................................................

Fix THttpServer to not call the cookie function with an empty cookie

This patch checks if the value passed in the 'Cookie' header to the
http hs2 server is blank, and if so it ignores it.

The reason to do this is so that a client sending an empty cookie
header isn't counted as a failed cookie attempt, which is incorrect.

Change-Id: I04e96fe97baae474a82fd30f2cd55ccce80570b4
---
M be/src/transport/THttpServer.cpp
1 file changed, 9 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/49/14149/2
--
To view, visit http://gerrit.cloudera.org:8080/14149
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I04e96fe97baae474a82fd30f2cd55ccce80570b4
Gerrit-Change-Number: 14149
Gerrit-PatchSet: 2
Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>
Gerrit-Reviewer: Attila Jeges <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>

Reply via email to