)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c5131299ccff808a6636d201255f0cd776398adf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6302433f_d49b8165","updated":"2024-10-16 10:53:23.000000000","message":"Thanks @Thomas \n\nI\u0027d like this bug fixed sooner rather than later - I ran into using parse_header on feature/mpu.\n\nI updated Tim\u0027s patch https://review.opendev.org/c/openstack/swift/+/887908 - I wonder if we can agree on that approach ?","commit_id":"849c644dd0180c9a8bce136fe96d66ea442a3b49"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"fffdb04ca73aed983642c9e83b3fbcb799d34eed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"81bb364b_74621ddf","updated":"2024-10-16 04:40:08.000000000","message":"Thanks, Tim also has one available (as always) which probably works better then email.message.Message, because that doesn\u0027t handle multiple parameters: https://review.opendev.org/c/openstack/swift/+/887908","commit_id":"849c644dd0180c9a8bce136fe96d66ea442a3b49"}],"swift/common/utils/parseh.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c5131299ccff808a6636d201255f0cd776398adf","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    # Adapted from https://peps.python.org/pep-0594/#cgi"},{"line_number":7,"context_line":"    m \u003d Message()"},{"line_number":8,"context_line":"    m[\u0027content-type\u0027] \u003d value"},{"line_number":9,"context_line":"    return dict(m.get_params())"}],"source_content_type":"text/x-python","patch_set":2,"id":"a68d6771_0b496f6a","line":9,"updated":"2024-10-16 10:53:23.000000000","message":"this is returning a dict but the caller is expecting a tuple\n\ncgi.parse_header doc says:\n\n```\nReturn the main content-type and a dictionary of options.\n```\n\nso you\u0027d need to unpack the params a bit more, something like\n\n```\nparams \u003d m.get_params()\nreturn params[0][0], dict(params[1:])\n```","commit_id":"849c644dd0180c9a8bce136fe96d66ea442a3b49"}]}
