Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/23153 )
Change subject: [REST API] Mapping Kerberos principals to local usernames ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/23153/1/src/kudu/master/rest_catalog_path_handlers.cc File src/kudu/master/rest_catalog_path_handlers.cc: http://gerrit.cloudera.org:8080/#/c/23153/1/src/kudu/master/rest_catalog_path_handlers.cc@91 PS1, Line 91: kudu::security::MapPrincipalToLocalName consider adding the necessary "using" directive at the beginning of the file http://gerrit.cloudera.org:8080/#/c/23153/1/src/kudu/master/rest_catalog_path_handlers.cc@263 PS1, Line 263: RETURN_JSON_ERROR(jw, : "Authentication principal could not be mapped to local user", : *status_code, : HttpStatusCode::Forbidden); Consider including the provided principal in the error message, it might be helpful for debugging. Here and in the next instances as well. -- To view, visit http://gerrit.cloudera.org:8080/23153 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib25a7886c32cbbef35272cd5799ae84601335a34 Gerrit-Change-Number: 23153 Gerrit-PatchSet: 1 Gerrit-Owner: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Mon, 14 Jul 2025 07:18:12 +0000 Gerrit-HasComments: Yes
