)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"d4458157fe71406902b6bc07d9d6b079367b5fdc","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Parameter to return audit ids only in revocation list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The revocation list will only include audit IDs and not token"},{"line_number":10,"context_line":"IDs in the revocation list if ?audit_id_only is on the"},{"line_number":11,"context_line":"request. Also, the data won\u0027t be obfuscated since the"},{"line_number":12,"context_line":"audit_ids aren\u0027t useful for auth. If the clients (typically"},{"line_number":13,"context_line":"auth_token) only request the revocation list with audit IDs"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9a68dd71_dc9668ef","line":10,"updated":"2016-01-23 03:53:36.000000000","message":"(nit) s/in the revocation list//\n\nNo need for the duplication.","commit_id":"4389abbc7afaad00c6a9f25d2b2ebe43283fbf01"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"76e1ac30437477c9a762060c64b8b4894353d787","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Parameter to return audit ids only in revocation list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The revocation list will only include audit IDs and not token"},{"line_number":10,"context_line":"IDs in the revocation list if ?audit_id_only is on the"},{"line_number":11,"context_line":"request. Also, the data won\u0027t be obfuscated since the"},{"line_number":12,"context_line":"audit_ids aren\u0027t useful for auth. If the clients (typically"},{"line_number":13,"context_line":"auth_token) only request the revocation list with audit IDs"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9a68dd71_e499217a","line":10,"in_reply_to":"9a68dd71_dc9668ef","updated":"2016-01-25 14:45:39.000000000","message":"Done","commit_id":"4389abbc7afaad00c6a9f25d2b2ebe43283fbf01"}],"keystone/tests/unit/test_v3_auth.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"75de6a03f3d243fad91451283907b258e42b19a6","unresolved":false,"context_lines":[{"line_number":4400,"context_line":"        res \u003d self.get(\u0027/auth/tokens/OS-PKI/revoked?audit_id_only\u0027)"},{"line_number":4401,"context_line":""},{"line_number":4402,"context_line":"        def truncate(ts_str):"},{"line_number":4403,"context_line":"            return ts_str[:19] + \u0027Z\u0027  # 2016-01-21T15:53:52 \u003d\u003d 19 chars."},{"line_number":4404,"context_line":""},{"line_number":4405,"context_line":"        exp_token_revoke_data \u003d {"},{"line_number":4406,"context_line":"            \u0027audit_id\u0027: token_data[\u0027audit_ids\u0027][0],"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a68dd71_8f1a107c","line":4403,"range":{"start_line":4403,"start_character":12,"end_line":4403,"end_character":35},"updated":"2016-01-22 15:23:48.000000000","message":"This is done because its done in the revoke api somewhere?","commit_id":"4389abbc7afaad00c6a9f25d2b2ebe43283fbf01"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"329b13c12f956e9ff4914c53e3740029c19f87f4","unresolved":false,"context_lines":[{"line_number":4400,"context_line":"        res \u003d self.get(\u0027/auth/tokens/OS-PKI/revoked?audit_id_only\u0027)"},{"line_number":4401,"context_line":""},{"line_number":4402,"context_line":"        def truncate(ts_str):"},{"line_number":4403,"context_line":"            return ts_str[:19] + \u0027Z\u0027  # 2016-01-21T15:53:52 \u003d\u003d 19 chars."},{"line_number":4404,"context_line":""},{"line_number":4405,"context_line":"        exp_token_revoke_data \u003d {"},{"line_number":4406,"context_line":"            \u0027audit_id\u0027: token_data[\u0027audit_ids\u0027][0],"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a68dd71_a6e051fb","line":4403,"range":{"start_line":4403,"start_character":12,"end_line":4403,"end_character":35},"in_reply_to":"9a68dd71_8f1a107c","updated":"2016-01-22 20:12:08.000000000","message":"It happens in keystone/auth/controllers.py revocation_list() where it does\n\n t[\u0027expires\u0027] \u003d utils.isotime(expires)\n\nIf it did utils.isotime(expires, subsecond\u003dTrue) then the timestamp wouldn\u0027t be truncated.","commit_id":"4389abbc7afaad00c6a9f25d2b2ebe43283fbf01"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"9061113065bc550e3d2cf380e16c5c3946d7ef79","unresolved":false,"context_lines":[{"line_number":4377,"context_line":""},{"line_number":4378,"context_line":"        self.assertEqual({\u0027revoked\u0027: [exp_token_revoke_data]}, payload)"},{"line_number":4379,"context_line":""},{"line_number":4380,"context_line":"    def test_audit_id_only_no_tokens(self):"},{"line_number":4381,"context_line":"        # When there\u0027s no revoked tokens and ?audit_id_only is used, the"},{"line_number":4382,"context_line":"        # response is an empty list and is not signed."},{"line_number":4383,"context_line":"        res \u003d self.get(\u0027/auth/tokens/OS-PKI/revoked?audit_id_only\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a5ae5dd_9b856808","line":4380,"updated":"2016-02-05 14:53:12.000000000","message":"nit: \n\n    def test_audit_id_only_with_no_tokens(self):","commit_id":"d1a980939b2057e2a117a7961f3cca4d6b56f258"}]}