)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"db65d6c6a969e4da0be785a2a2ef38c590b6beb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a79026a3_48072527","updated":"2024-03-06 18:31:58.000000000","message":"LGTM, thanks!","commit_id":"cc2778004205d6a42d9a3f01bed8f21af0b482d2"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"213889df25c4e4bb99fe8bdc7700c12a04d2108f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5f73e636_3ab84adc","updated":"2024-03-08 10:38:03.000000000","message":"recheck","commit_id":"cc2778004205d6a42d9a3f01bed8f21af0b482d2"}],"doc/source/development_middleware.rst":[{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"41189e8d864c6dfe24a38db63828535037c90863","unresolved":true,"context_lines":[{"line_number":323,"context_line":"Object immutable metadata includes::"},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"    X-Timestamp"},{"line_number":326,"context_line":"    Content-Length"},{"line_number":327,"context_line":"    Etag"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"``X-Timestamp`` and ``Content-Length`` metadata MUST be included in PUT"}],"source_content_type":"text/x-rst","patch_set":2,"id":"81f80ab3_2b8adc45","line":326,"updated":"2024-03-06 18:26:28.000000000","message":"should Content-Length be removed from list of immutable metadata, since POST can modify it?","commit_id":"cc2778004205d6a42d9a3f01bed8f21af0b482d2"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"329d8d8541a23689e2899f2b95c1f61f44e1b302","unresolved":true,"context_lines":[{"line_number":323,"context_line":"Object immutable metadata includes::"},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"    X-Timestamp"},{"line_number":326,"context_line":"    Content-Length"},{"line_number":327,"context_line":"    Etag"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"``X-Timestamp`` and ``Content-Length`` metadata MUST be included in PUT"}],"source_content_type":"text/x-rst","patch_set":2,"id":"96c1572e_f0bce7cb","line":326,"in_reply_to":"81f80ab3_2b8adc45","updated":"2024-03-06 18:29:20.000000000","message":"Content-Length cannot be modified by a POST","commit_id":"cc2778004205d6a42d9a3f01bed8f21af0b482d2"},{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"db65d6c6a969e4da0be785a2a2ef38c590b6beb7","unresolved":false,"context_lines":[{"line_number":323,"context_line":"Object immutable metadata includes::"},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"    X-Timestamp"},{"line_number":326,"context_line":"    Content-Length"},{"line_number":327,"context_line":"    Etag"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"``X-Timestamp`` and ``Content-Length`` metadata MUST be included in PUT"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bc35afed_ff644892","line":326,"in_reply_to":"96c1572e_f0bce7cb","updated":"2024-03-06 18:31:58.000000000","message":"my bad, I got confused with ``Content-Type``.","commit_id":"cc2778004205d6a42d9a3f01bed8f21af0b482d2"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"28fc193d1f5ebd759a874cb6739ee3006481e589","unresolved":true,"context_lines":[{"line_number":331,"context_line":"when they handle a PUT request, but checked against any ``Etag`` header sent"},{"line_number":332,"context_line":"with the PUT request."},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"Object immutable metadata, along with ``Content-Type``, is the only object"},{"line_number":335,"context_line":"metadata that is stored by container servers and returned in object listings."},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"************"},{"line_number":338,"context_line":"Content-Type"}],"source_content_type":"text/x-rst","patch_set":2,"id":"d650a82b_c3982067","line":335,"range":{"start_line":334,"start_character":59,"end_line":335,"end_character":76},"updated":"2024-03-06 16:48:10.000000000","message":"Well.... kind of. But maybe [our use of `x-backend-container-update-override-etag`](https://github.com/openstack/swift/blob/master/swift/common/middleware/s3api/controllers/multi_upload.py#L768-L770) (and the `x-backend-container-update-override-*` namespace generally) should remain out of scope for this patch 😊\n\nI *do* want to [expand its use](https://review.opendev.org/c/openstack/swift/+/909802/5/swift/common/middleware/s3api/s3request.py#915), though.","commit_id":"cc2778004205d6a42d9a3f01bed8f21af0b482d2"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"0f840e89273e687346c15314e94eb45cb8479928","unresolved":true,"context_lines":[{"line_number":331,"context_line":"when they handle a PUT request, but checked against any ``Etag`` header sent"},{"line_number":332,"context_line":"with the PUT request."},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"Object immutable metadata, along with ``Content-Type``, is the only object"},{"line_number":335,"context_line":"metadata that is stored by container servers and returned in object listings."},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"************"},{"line_number":338,"context_line":"Content-Type"}],"source_content_type":"text/x-rst","patch_set":2,"id":"79195293_f68e924c","line":335,"range":{"start_line":334,"start_character":59,"end_line":335,"end_character":76},"in_reply_to":"d650a82b_c3982067","updated":"2024-03-06 17:40:59.000000000","message":"😮 oh goodness yes! it\u0027s like peeling an onion in more than one way:\n\n(a) turns out there\u0027s more layers than you think\n\n(b) makes you cry","commit_id":"cc2778004205d6a42d9a3f01bed8f21af0b482d2"}]}