)]}'
{
  "commit": "42c848056bb4534a095c2531c3452d663d318afd",
  "tree": "069b81d85accd7a048dbd4e0cc80c74285b6779b",
  "parents": [
    "de7fe329140ea99e6b791d1f6488ae68732b8b7f"
  ],
  "author": {
    "name": "Luca Milanesio",
    "email": "luca.milanesio@gmail.com",
    "time": "Fri Feb 07 12:42:00 2025 +0000"
  },
  "committer": {
    "name": "Luca Milanesio",
    "email": "luca.milanesio@gmail.com",
    "time": "Fri Feb 07 13:08:06 2025 +0000"
  },
  "message": "Fix RepoView leak in CodeOwnerConfigValidator\n\nThe onPreMerge() operation of CodeOwnerConfigValidator was creating\na RepoView on-the-fly without a proper try-with-resource causing\nthe leak of a Repository object reference counting.\n\nExtract the repoView creation in a separate variable and make sure\nit is included in the outer try-with-resource everywhere it is used.\n\nNOTE: ErrorProne should have spotted resources leaks in the\nfirst place, so we should consider enabling the MustBeClosedChecker.\n\n[1] https://errorprone.info/bugpattern/MustBeClosedChecker\n\nChange-Id: I2880a4cf8b76e2be979288c996720478f98b4fba\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "84115680368d97a0cb74bf7156d0f495144602a4",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java",
      "new_id": "1a081315ce81049046eab00d59f5afdb4d3fa5d4",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java"
    },
    {
      "type": "modify",
      "old_id": "c3dece01bbc2034c9410a2915fdef959c81e23f9",
      "old_mode": 33188,
      "old_path": "javatests/com/google/gerrit/plugins/codeowners/backend/config/CodeOwnersPluginConfigValidatorTest.java",
      "new_id": "0957119486c004f7f2996bbd293f02371636d3a0",
      "new_mode": 33188,
      "new_path": "javatests/com/google/gerrit/plugins/codeowners/backend/config/CodeOwnersPluginConfigValidatorTest.java"
    }
  ]
}