)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"0bde86109d456620b5bbcbb620e1e556d1d046b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"463ecc4a_5e2deb9c","updated":"2023-08-16 18:28:41.000000000","message":"this patch came up while reviewing https://review.opendev.org/c/openstack/swift/+/891526/1\n\nin a broader context it makes me question how many of the other close_if_possible could perhaps benifit from reading some of the (mostly small/error?) bodies.","commit_id":"12d8a53fffea6e4bed8ba3d502ce625f5c6710b9"}],"swift/common/swob.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"0bde86109d456620b5bbcbb620e1e556d1d046b7","unresolved":true,"context_lines":[{"line_number":1257,"context_line":"                return [\u0027\u0027]"},{"line_number":1258,"context_line":""},{"line_number":1259,"context_line":"        if self.request and self.request.method \u003d\u003d \u0027HEAD\u0027:"},{"line_number":1260,"context_line":"            # We explicitly do NOT want to set self.content_length to 0 here"},{"line_number":1261,"context_line":"            return [\u0027\u0027]"},{"line_number":1262,"context_line":""},{"line_number":1263,"context_line":"        if self.conditional_response and self.request and \\"}],"source_content_type":"text/x-python","patch_set":2,"id":"82af0fd9_7c2908f4","line":1260,"updated":"2023-08-16 18:28:41.000000000","message":"very curious why not close here; possibly also some concern that \"close_and_drain\" should maybe only read a few MB max before giving up and just letting it backend 499 if it wants to.","commit_id":"12d8a53fffea6e4bed8ba3d502ce625f5c6710b9"}]}