)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"02976ed686c136521bbe706705cb69ee6dee89d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"04c7a0ab_df5daca7","updated":"2025-03-04 17:25:52.000000000","message":"Needs plumbing in `test/functional/s3api/test_xxe_injection.py` (which uses presigned URLs and `requests`)\n\nCould also use some plumbing over in `test/s3api/`","commit_id":"9fc9629e6966b343fdd754b99f1378492079912e"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"f2d1ea12d765e39193f1a70e2c08cbbacd363cbb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"e9c27c29_26121287","updated":"2025-04-09 23:46:04.000000000","message":"Even if it\u0027s incomplete, it\u0027s a damn sight better.","commit_id":"6ba472c4c9802569ef6f40972e5ea74c9e6816e7"}],"test/functional/swift_test_client.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"b7a8c399f6dd3d61b7469c93c7cc4678fb55cd08","unresolved":false,"context_lines":[{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    def http_connect(self):"},{"line_number":225,"context_line":"        self.connection \u003d self.conn_class(self.storage_host,"},{"line_number":226,"context_line":"                                          port\u003dself.storage_port)"},{"line_number":227,"context_line":"        # self.connection.set_debuglevel(3)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"    def make_path(self, path\u003dNone, cfg\u003dNone):"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_fda72da7","line":226,"updated":"2018-06-11 23:16:30.000000000","message":"I\u0027m surprised this didn\u0027t need to change...","commit_id":"90327ff64c8d08347f94daf270082bdee2faf1d4"}],"test/s3api/test_input_errors.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"8f1649c6d3cce396564e9b3383c653b5e501a558","unresolved":true,"context_lines":[{"line_number":692,"context_line":"        host, port \u003d parse_socket_string(request[\u0027host\u0027], None)"},{"line_number":693,"context_line":"        if port:"},{"line_number":694,"context_line":"            port \u003d int(port)"},{"line_number":695,"context_line":"        conn \u003d bufferedhttp.http_connect_raw("},{"line_number":696,"context_line":"            host,"},{"line_number":697,"context_line":"            port,"},{"line_number":698,"context_line":"            request[\u0027method\u0027],"}],"source_content_type":"text/x-python","patch_set":13,"id":"ccf7e342_dcc5ccfd","line":695,"updated":"2025-03-04 20:04:23.000000000","message":"These `_no_content_length` tests will also need updating, but that\u0027s going to require some plumbing into `bufferedhttp` which seems somewhat involved. Given the goal of the chain is to get TLS enabled for our DSVM jobs and those don\u0027t [(yet)](https://review.opendev.org/c/openstack/swift/+/942133) involve our s3api tests, I\u0027m inclined to leave it for now.","commit_id":"d163280152fc62827b99105cf91feb331f762744"}],"test/sample.conf":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"85ce0cb8eb77fdd084b31f904ced78b0ee7ea4c4","unresolved":true,"context_lines":[{"line_number":25,"context_line":"#auth_uri \u003d http://localhost:5000/v3/"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"# Set this path if you have a custom .pem for cert validation"},{"line_number":28,"context_line":"#cafile \u003d /opt/stack/data/ca-bundle.pem"},{"line_number":29,"context_line":"# Used by s3api functional tests, which don\u0027t contact auth directly"},{"line_number":30,"context_line":"#s3_storage_url \u003d http://127.0.0.1:8080/"},{"line_number":31,"context_line":"#s3_region \u003d us-east-1"}],"source_content_type":"text/plain","patch_set":12,"id":"83696af5_77a3768f","line":28,"updated":"2025-03-04 19:30:58.000000000","message":"This would maybe be better as `ca_cert` to match the option name already present in `[s3api_test]` ... but devstack [already has plumbing for `cafile`](https://github.com/openstack/devstack/blob/db5e05ac/lib/swift#L538) :-/","commit_id":"9fc9629e6966b343fdd754b99f1378492079912e"}]}