)]}'
{
  "commit": "3a06b4d609a514aff792d6c3cf033a3dedb42edb",
  "tree": "53692975418870224b3d8d9531edaded130c3859",
  "parents": [
    "28fb620851d34928af50b10b6adba323c347180b"
  ],
  "author": {
    "name": "diannahu",
    "email": "diannahu@google.com",
    "time": "Wed May 03 12:27:18 2023 -0700"
  },
  "committer": {
    "name": "Copybara-Service",
    "email": "copybara-worker@google.com",
    "time": "Wed May 03 12:28:19 2023 -0700"
  },
  "message": "Avoid interim response handling for 101 Switching Protocols responses in BalsaFrame.\n\nWhen updating SimpleHttpClient to set_use_interim_headers_callback(true) in\ncl/527686928, I found that the client would not receive WebSocket payloads as\nbody after receiving the 101 response in some cases. This was likely because of\nthe state resetting in [1]. The resetting is consistent with 100 Continue\nbehavior and will likely also be helpful with 103 Early Hints (and other\ninterim responses), but is not helpful for Upgrade/101 Switching Protocols.\n\nThis CL special-cases 101 Switching Protocols, excluding these responses from\nother interim response handling (involving OnInterimHeaders()), because 101\nrepresents the \"final\" headers on the current protocol.\n\nThe added unit test fails without this change (missing, headers callback aside,\nthe body callbacks. Balsa instead consumes the entire message but only outputs\nthe header portion. Perhaps the body was erroneously considered as the start of\n\"final\" headers).\n\n[1] http://google3/third_party/quiche/balsa/balsa_frame.cc;l\u003d857;rcl\u003d524360383\n\nPiperOrigin-RevId: 529163242\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "ac5e018528e9efd336bfa700e584af410ab7a78b",
      "old_mode": 33188,
      "old_path": "quiche/balsa/balsa_frame.cc",
      "new_id": "592a4db4e3ad799233d6cceb7f580e0b8e2d8fcf",
      "new_mode": 33188,
      "new_path": "quiche/balsa/balsa_frame.cc"
    },
    {
      "type": "modify",
      "old_id": "13a704a2c0003b8d3ddb091f3df54e86a6b897ac",
      "old_mode": 33188,
      "old_path": "quiche/balsa/balsa_frame_test.cc",
      "new_id": "83619d4f322d55358d98ee92d7b99a0106b27d1e",
      "new_mode": 33188,
      "new_path": "quiche/balsa/balsa_frame_test.cc"
    }
  ]
}
