)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10234,"name":"Andrey Pavlov","email":"andrey.mp@gmail.com","username":"andrey-mp"},"change_message_id":"2be9a10d05cda1793b57bb8297c1f7a29db75ee2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c37390fd_d3ed8cb5","updated":"2022-04-04 05:38:20.000000000","message":"recheck","commit_id":"ed85f5924cc0687a42032dcf363081a8c224032b"}],"ec2api/api/faults.py":[{"author":{"_account_id":10234,"name":"Andrey Pavlov","email":"andrey.mp@gmail.com","username":"andrey-mp"},"change_message_id":"c2d24069b4b09122b396876f423a262444b1f4c9","unresolved":true,"context_lines":[{"line_number":65,"context_line":"            raise webob.exc.HTTPBadRequest()"},{"line_number":66,"context_line":"        user_id, _sep, project_id \u003d req.params[\u0027AWSAccessKeyId\u0027].partition(\u0027:\u0027)"},{"line_number":67,"context_line":"        project_id \u003d project_id or user_id"},{"line_number":68,"context_line":"        remote_address \u003d getattr(req, \u0027remote_addr\u0027, \u0027127.0.0.1\u0027)"},{"line_number":69,"context_line":"        if CONF.use_forwarded_for:"},{"line_number":70,"context_line":"            remote_address \u003d req.headers.get(\u0027X-Forwarded-For\u0027, remote_address)"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"8c01f38d_c66ef3db","line":68,"updated":"2022-04-03 18:42:59.000000000","message":"looks strange - we define remote_address here but don\u0027t use it...\nI would suggest to remove it at all.","commit_id":"6c05816ffd7dae4355de5c1334101f8809455385"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8397265fb54b95a5a2f5dac7e5b3a5909b64b002","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            raise webob.exc.HTTPBadRequest()"},{"line_number":66,"context_line":"        user_id, _sep, project_id \u003d req.params[\u0027AWSAccessKeyId\u0027].partition(\u0027:\u0027)"},{"line_number":67,"context_line":"        project_id \u003d project_id or user_id"},{"line_number":68,"context_line":"        remote_address \u003d getattr(req, \u0027remote_addr\u0027, \u0027127.0.0.1\u0027)"},{"line_number":69,"context_line":"        if CONF.use_forwarded_for:"},{"line_number":70,"context_line":"            remote_address \u003d req.headers.get(\u0027X-Forwarded-For\u0027, remote_address)"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9fa293c8_a014d981","line":68,"in_reply_to":"8c01f38d_c66ef3db","updated":"2022-04-03 23:10:15.000000000","message":"Oh, good catch. I\u0027ve also removed L66 and L67 because user_id and project_id are unused, too.","commit_id":"6c05816ffd7dae4355de5c1334101f8809455385"}]}