Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/24545 )
Change subject: KUDU-3777: return HTTP 401 for bad SPNEGO headers ...................................................................... Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/24545/6/src/kudu/server/webserver-test.cc File src/kudu/server/webserver-test.cc: http://gerrit.cloudera.org:8080/#/c/24545/6/src/kudu/server/webserver-test.cc@a550 PS6, Line 550: Why was this assertion removed? I just tried locally, and the test works fine if you add back the assertion. http://gerrit.cloudera.org:8080/#/c/24545/6/src/kudu/server/webserver.cc File src/kudu/server/webserver.cc: http://gerrit.cloudera.org:8080/#/c/24545/6/src/kudu/server/webserver.cc@713 PS6, Line 713: — nit: please avoid unicode chars in the codebase. -- 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: comment Gerrit-Change-Id: Ie1e85a991c50cddeec76a2ab16a65b34fe3fee5c Gerrit-Change-Number: 24545 Gerrit-PatchSet: 6 Gerrit-Owner: Jeison Romero <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Jeison Romero <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Comment-Date: Wed, 08 Jul 2026 11:35:02 +0000 Gerrit-HasComments: Yes
