)]}'
{
  "commit": "2c47668e9e166db5d94df8c1f915b956942e9c9d",
  "tree": "3024bc94b5ea488e609d8528ab65ab68ca091d7e",
  "parents": [
    "20eacd4c8afaa0efe62476c25a796abae787421d"
  ],
  "author": {
    "name": "Edwin Kempin",
    "email": "ekempin@google.com",
    "time": "Mon Dec 08 15:13:52 2025 +0000"
  },
  "committer": {
    "name": "Edwin Kempin",
    "email": "ekempin@google.com",
    "time": "Mon Dec 08 15:13:52 2025 +0000"
  },
  "message": "SuperManifestRefUpdatedListener: Do not return 500 via Response.withStatusCode\n\nREST endpoints should throw an exception to indicate an internal server\nerror. RestApiServlet takes care to catch the exception and to return a\n\u0027500 Internal Server Error\u0027 response for it.\n\nReturning 500 via Response.withStatusCode also works but messes up the\nerror metric. If no exception is thrown, then we can\u0027t log an exception\nas the cause in the error metric and we fall back to using \"_none\" as\nthe cause (hard-coded in RestApiServlet). \"_none\" is not helpful, hence\nfix the metric by throwing an exception, rather than returning a 500\nresponse code with Response.withStatusCode.\n\nChange-Id: Ic479a221985bf5d936fc88def24837d343c73dd5\nSigned-off-by: Edwin Kempin \u003cekempin@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "b2322147cc36340fd573c1040f18771cec28b3be",
      "old_mode": 33188,
      "old_path": "java/com/googlesource/gerrit/plugins/supermanifest/SuperManifestRefUpdatedListener.java",
      "new_id": "efa55771fe803150802fae8e30dd3b06cb109ee0",
      "new_mode": 33188,
      "new_path": "java/com/googlesource/gerrit/plugins/supermanifest/SuperManifestRefUpdatedListener.java"
    }
  ]
}