)]}'
{"keystone/tests/unit/test_auth.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"d856ae0318b4e1db029db753fb47af044fd1637d","unresolved":false,"context_lines":[{"line_number":687,"context_line":"                          body_dict)"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"    def test_revoke_with_no_audit_info(self):"},{"line_number":690,"context_line":"        self.skipTest(\u0027Fernet with v2.0 and revocation is broken\u0027)"},{"line_number":691,"context_line":""},{"line_number":692,"context_line":"    def test_deleting_role_revokes_token(self):"},{"line_number":693,"context_line":"        self.skipTest(\u0027Fernet with v2.0 and revocation is broken\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_b43c95c0","line":690,"updated":"2016-05-02 23:38:23.000000000","message":"we can probably document this in another bug if it hasn\u0027t be done already.","commit_id":"a63aaf2abf8c2733e150dac0cade575702e102e9"}],"keystone/token/providers/fernet/token_formatters.py":[{"author":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"change_message_id":"4190d75c353d3c5660591a4a56f1060de8469f0a","unresolved":false,"context_lines":[{"line_number":450,"context_line":"            # uuid for project IDs, instead they use non-uuid strings. This"},{"line_number":451,"context_line":"            # pattern could probably be applied to the rest of this formatter,"},{"line_number":452,"context_line":"            # but this is the only break that was exposed when testing fernet +"},{"line_number":453,"context_line":"            # v2.0 + non-uuid project IDs."},{"line_number":454,"context_line":"            project_id \u003d project_id.decode(\"utf-8\")"},{"line_number":455,"context_line":"        expires_at_str \u003d cls._convert_float_to_time_string(payload[3])"},{"line_number":456,"context_line":"        audit_ids \u003d list(map(provider.base64_encode, payload[4]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_249afbc5","line":453,"updated":"2016-05-03 17:40:26.000000000","message":"So, this patch doesn\u0027t fix this issue? https://review.openstack.org/#/c/306681/\n\nOr should we do some similar for this case?","commit_id":"fca1657de2cd3795fd60322435549f8b9e3ee77f"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"9630d1ccb1158281d99f9c117294e59ba02660b3","unresolved":false,"context_lines":[{"line_number":450,"context_line":"            # uuid for project IDs, instead they use non-uuid strings. This"},{"line_number":451,"context_line":"            # pattern could probably be applied to the rest of this formatter,"},{"line_number":452,"context_line":"            # but this is the only break that was exposed when testing fernet +"},{"line_number":453,"context_line":"            # v2.0 + non-uuid project IDs."},{"line_number":454,"context_line":"            project_id \u003d project_id.decode(\"utf-8\")"},{"line_number":455,"context_line":"        expires_at_str \u003d cls._convert_float_to_time_string(payload[3])"},{"line_number":456,"context_line":"        audit_ids \u003d list(map(provider.base64_encode, payload[4]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_4411f703","line":453,"in_reply_to":"1a122d0e_249afbc5","updated":"2016-05-03 17:47:55.000000000","message":"Raildo, it looks like Adam\u0027s patch fixes this but in a different way. I would be fine taking both approaches. The first would be correcting the test like what Adam is doing, and the second we could do something like this where we make sure the project_id is always of type str before returning if from the formatter.","commit_id":"fca1657de2cd3795fd60322435549f8b9e3ee77f"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"7af196361f03bddac6729c948b71d23add0f8fab","unresolved":false,"context_lines":[{"line_number":450,"context_line":"            # uuid for project IDs, instead they use non-uuid strings. This"},{"line_number":451,"context_line":"            # pattern could probably be applied to the rest of this formatter,"},{"line_number":452,"context_line":"            # but this is the only break that was exposed when testing fernet +"},{"line_number":453,"context_line":"            # v2.0 + non-uuid project IDs."},{"line_number":454,"context_line":"            project_id \u003d project_id.decode(\"utf-8\")"},{"line_number":455,"context_line":"        expires_at_str \u003d cls._convert_float_to_time_string(payload[3])"},{"line_number":456,"context_line":"        audit_ids \u003d list(map(provider.base64_encode, payload[4]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_501fe808","line":453,"in_reply_to":"1a122d0e_4411f703","updated":"2016-05-03 22:07:49.000000000","message":"https://review.openstack.org/#/c/306681/ is now gating, so if this can be removed then go ahead.","commit_id":"fca1657de2cd3795fd60322435549f8b9e3ee77f"}]}