)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"662666209ff7358bc593db48495e35db66f11c3e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Support format for msgpack \u003c 1.0 in token formatter"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"msgpack v1.0 changed its data format [1] and during a rolling upgrade, attempts"},{"line_number":10,"context_line":"to unpack cached tokens with old data format with the new default raw\u003dFalse"},{"line_number":11,"context_line":"result in the following error:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"  UnicodeDecodeError: \u0027utf-8\u0027 codec can\u0027t decode byte 0x87 in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_ef01bc3b","line":10,"range":{"start_line":10,"start_character":10,"end_line":10,"end_character":25},"updated":"2020-08-12 08:28:05.000000000","message":"the alternitive fix would be to catch the deserialiseation error and then discard the token and revive a new one instead of a cached token.","commit_id":"7d6c71ba26694c21110280e741b9ffe2d36a94ca"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"662666209ff7358bc593db48495e35db66f11c3e","unresolved":false,"context_lines":[{"line_number":10,"context_line":"to unpack cached tokens with old data format with the new default raw\u003dFalse"},{"line_number":11,"context_line":"result in the following error:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"  UnicodeDecodeError: \u0027utf-8\u0027 codec can\u0027t decode byte 0x87 in"},{"line_number":14,"context_line":"  position 3: invalid start byte"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This passes raw\u003dTrue to support backward-compat with the old format"},{"line_number":17,"context_line":"until we are guaranteed to have msgpack \u003e\u003d 1.0 in the N-1 release of"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_6f93ec01","line":14,"range":{"start_line":13,"start_character":1,"end_line":14,"end_character":32},"updated":"2020-08-12 08:28:05.000000000","message":"so this looks like a byte string vs text string conversion error.\n\nthey were using raw byte stings before and now they are using utf-8 text strings.","commit_id":"7d6c71ba26694c21110280e741b9ffe2d36a94ca"}],"keystone/token/token_formatters.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"662666209ff7358bc593db48495e35db66f11c3e","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        serialized_payload \u003d self.unpack(token)"},{"line_number":173,"context_line":"        # TODO(melwitt): msgpack changed their data format in version 1.0, so"},{"line_number":174,"context_line":"        # in order to support a rolling upgrade, we must pass raw\u003dTrue to"},{"line_number":175,"context_line":"        # to support the old format. The try-except may be removed in the once"},{"line_number":176,"context_line":"        # the N-1 release no longer supports msgpack \u003c 1.0."},{"line_number":177,"context_line":"        try:"},{"line_number":178,"context_line":"            versioned_payload \u003d msgpack.unpackb(serialized_payload)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_4ff38874","line":175,"range":{"start_line":175,"start_character":66,"end_line":175,"end_character":74},"updated":"2020-08-12 08:28:05.000000000","message":"nit: delete","commit_id":"7d6c71ba26694c21110280e741b9ffe2d36a94ca"}]}