)]}'
{"keystone/tests/unit/test_v3_auth.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"d8c67e8025b15f2eeccb9debe73cb89f7de48e26","unresolved":false,"context_lines":[{"line_number":4091,"context_line":"        return self._make_auth_request(auth_data)"},{"line_number":4092,"context_line":""},{"line_number":4093,"context_line":"    def _validate_token(self, token, expected_status\u003d200):"},{"line_number":4094,"context_line":"        # Use the admin token to validate the specified token."},{"line_number":4095,"context_line":"        return self.get("},{"line_number":4096,"context_line":"            \u0027/auth/tokens\u0027,"},{"line_number":4097,"context_line":"            token\u003d\u0027ADMIN\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_adf6b7f7","line":4094,"updated":"2015-03-17 00:33:53.000000000","message":"we can see that... the question is why?","commit_id":"fc8e1d1ace971dfdcc2b5d5dbe86c058504cff2c"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"41619f18fa68e007d9378fdea54c7b490eb1d687","unresolved":false,"context_lines":[{"line_number":4091,"context_line":"        return self._make_auth_request(auth_data)"},{"line_number":4092,"context_line":""},{"line_number":4093,"context_line":"    def _validate_token(self, token, expected_status\u003d200):"},{"line_number":4094,"context_line":"        # Use the admin token to validate the specified token."},{"line_number":4095,"context_line":"        return self.get("},{"line_number":4096,"context_line":"            \u0027/auth/tokens\u0027,"},{"line_number":4097,"context_line":"            token\u003d\u0027ADMIN\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_cdf5ab85","line":4094,"in_reply_to":"9a80dd14_adf6b7f7","updated":"2015-03-17 00:53:08.000000000","message":"I will add an explanation here.\n\nThe reason behind this is: if we do not specify a token to be used as \u0027X-Auth-Token\u0027 for the specified request, a token with self.user on self.project will be requested [1][2].\n\nSince some tests disable this user, change its password, etc this \u0027X-Auth-Token\u0027 cannot be retrieved and a 401 will be raised, even before trying to validate the \u0027X-Subject-Token\u0027 in question.\n\n[1] https://github.com/openstack/keystone/blob/master/keystone/tests/unit/test_v3.py#L460-L467\n\n[2] https://github.com/openstack/keystone/blob/master/keystone/tests/unit/test_v3.py#L408-L434","commit_id":"fc8e1d1ace971dfdcc2b5d5dbe86c058504cff2c"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"d8c67e8025b15f2eeccb9debe73cb89f7de48e26","unresolved":false,"context_lines":[{"line_number":4364,"context_line":"                          trust_scoped_token)"},{"line_number":4365,"context_line":"        self._validate_token(trust_scoped_token, expected_status\u003d404)"},{"line_number":4366,"context_line":""},{"line_number":4367,"context_line":"    @test_utils.wip(\u0027Waiting on bug #1432892\u0027)"},{"line_number":4368,"context_line":"    def test_trust_scoped_token_is_invalid_after_disabling_trustor(self):"},{"line_number":4369,"context_line":"        trustee_user, trust \u003d self._create_trust()"},{"line_number":4370,"context_line":"        trust_scoped_token \u003d self._get_trust_scoped_token(trustee_user, trust)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_4dce1bb9","line":4367,"updated":"2015-03-17 00:33:53.000000000","message":"this all worked before and now it\u0027s broken?","commit_id":"fc8e1d1ace971dfdcc2b5d5dbe86c058504cff2c"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"41619f18fa68e007d9378fdea54c7b490eb1d687","unresolved":false,"context_lines":[{"line_number":4364,"context_line":"                          trust_scoped_token)"},{"line_number":4365,"context_line":"        self._validate_token(trust_scoped_token, expected_status\u003d404)"},{"line_number":4366,"context_line":""},{"line_number":4367,"context_line":"    @test_utils.wip(\u0027Waiting on bug #1432892\u0027)"},{"line_number":4368,"context_line":"    def test_trust_scoped_token_is_invalid_after_disabling_trustor(self):"},{"line_number":4369,"context_line":"        trustee_user, trust \u003d self._create_trust()"},{"line_number":4370,"context_line":"        trust_scoped_token \u003d self._get_trust_scoped_token(trustee_user, trust)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_cd23eb04","line":4367,"in_reply_to":"9a80dd14_4dce1bb9","updated":"2015-03-17 00:53:08.000000000","message":"Yes, it is broken now because of the new assertion added at the end of this test.\n\nPlease see the bug #1432892 description for more info on why it is broken.\n\nI can add a doc here explaining the main cause of this skip.","commit_id":"fc8e1d1ace971dfdcc2b5d5dbe86c058504cff2c"}]}
