)]}' { "commit": "725b9b3be653ce0517b473d2c682bfe7e1ddadb5", "tree": "74bad8386ebaa792250c05b2365f8d8661d4233c", "parents": [ "bd05f014fefdab61a7bc87273489e8615f37397e" ], "author": { "name": "Youssef Elghareeb", "email": "ghareeb@google.com", "time": "Tue Jan 31 13:57:51 2023 +0100" }, "committer": { "name": "Youssef Elghareeb", "email": "ghareeb@google.com", "time": "Tue Jan 31 14:01:27 2023 +0100" }, "message": "ChecksUpdate: use ChangeIndexer to index changes\n\nWe saw in the logs thrown exceptions on this code path with message:\n * AuthException: maintain server not permitted\n\nThis was because the current code delegated to ChangeApiImpl which calls\nIndex (see [1]), which in turn checks that the calling user has the\n\u0027Maintain Server\u0027 global capability and fails. This failed the POST\nrequest to update checks.\n\nUpdating the code to use ChangeIndexer instead.\n\n[1] https://gerrit.googlesource.com/gerrit/+/88f26343b13560d4c75b25a77a59427972c1c91a/java/com/google/gerrit/server/restapi/change/Index.java#44\n\nGoogle-Bug-Id: b/250877007\nChange-Id: Iea638a383ecfc7ddf6a3b46ccc6a005eb5cd3aea\n", "tree_diff": [ { "type": "modify", "old_id": "336a961f95d51b787c302a85211dc77ea7124304", "old_mode": 33188, "old_path": "java/com/google/gerrit/plugins/checks/ChecksUpdate.java", "new_id": "0dae9a797093405908478c7eabb4afa4a333efb8", "new_mode": 33188, "new_path": "java/com/google/gerrit/plugins/checks/ChecksUpdate.java" } ] }