Jeison Romero has uploaded this change for review. ( http://gerrit.cloudera.org:8080/24545
Change subject: KUDU-3777: return HTTP 401 for bad SPNEGO headers ...................................................................... KUDU-3777: return HTTP 401 for bad SPNEGO headers When a client sends an Authorization header with an unrecognized scheme (e.g., Bearer) to a SPNEGO-protected webserver endpoint, RunSpnegoStep returns Status::InvalidArgument. Previously, the error dispatch in Webserver::BeginRequestCallback only mapped IsNotAuthorized() to HTTP 401, allowing InvalidArgument to fall through to an HTTP 500. This patch updates the error dispatch to map IsInvalidArgument() to HTTP 401 (Authentication Required), ensuring the correct client error is returned. The SpnegoWebserverTest.TestInvalidHeaders test has also been updated to expect the correct 401 response. Change-Id: Ie1e85a991c50cddeec76a2ab16a65b34fe3fee5c --- M src/kudu/server/webserver-test.cc M src/kudu/server/webserver.cc 2 files changed, 3 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/24545/1 -- To view, visit http://gerrit.cloudera.org:8080/24545 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ie1e85a991c50cddeec76a2ab16a65b34fe3fee5c Gerrit-Change-Number: 24545 Gerrit-PatchSet: 1 Gerrit-Owner: Jeison Romero <[email protected]>
