)]}'
{
  "commit": "e2c24a424947af59d6914b05f4093d12bff642bb",
  "tree": "bdced4ad255ba034849cbc4caddb7a1743639091",
  "parents": [
    "10bdfbdf73d679a7735a7d8087518acf1769c2bf"
  ],
  "author": {
    "name": "Youssef Elghareeb",
    "email": "ghareeb@google.com",
    "time": "Mon Oct 18 15:36:39 2021 +0200"
  },
  "committer": {
    "name": "Youssef Elghareeb",
    "email": "ghareeb@google.com",
    "time": "Mon Oct 18 14:42:53 2021 +0000"
  },
  "message": "Fix NPE in uploadvalidator\n\nChange I4081375e033 migrated uploadvalidator to use DiffOperations. The\nimplementation uses two different sources:\n1) CommitUtils#getChangedContent to get the changed files (regular,\nexecutable) and their content.\n2) DiffOperations (previously PatchListCache) to get the list of edits.\n\nWe loop on changed files using the file paths of [1], but we check the\nedits from [2]. We need to check if the file path exists in\nDiffOperation\u0027s output map. Sometimes DiffOperations skips some files,\nfor example if files that were due to rebase.\n\nBug: Google b/203216894\nChange-Id: Ib6dce965de5a2e79b28e1a6f76c36c8049734020\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "a229ee6347c025e14b605480a1ca71d50cdda729",
      "old_mode": 33188,
      "old_path": "src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/BlockedKeywordValidator.java",
      "new_id": "20fa0f8ca647e8699003b156ca47bfeaab86d8cd",
      "new_mode": 33188,
      "new_path": "src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/BlockedKeywordValidator.java"
    },
    {
      "type": "modify",
      "old_id": "c4d0ac4b593a9682bf95fd3ff10e8bff35fd58fb",
      "old_mode": 33188,
      "old_path": "src/test/java/com/googlesource/gerrit/plugins/uploadvalidator/BlockedKeywordValidatorTest.java",
      "new_id": "96d7e9043c171a26923a7abcf2fbc65b387b156f",
      "new_mode": 33188,
      "new_path": "src/test/java/com/googlesource/gerrit/plugins/uploadvalidator/BlockedKeywordValidatorTest.java"
    }
  ]
}
