)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"e5e66a73bcb68b5db3e46d2efec79671b4404e66","unresolved":false,"context_lines":[{"line_number":16,"context_line":"when a domain is disabled."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: If3e941018d5c2c9bd22397e69f83b7bf92643340"},{"line_number":19,"context_line":"Partial-Bug: 1532280"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1aa78d24_cb855508","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":7},"updated":"2016-07-07 20:55:14.000000000","message":"Closes?","commit_id":"d53db1889e17d493202743246243936af90234b9"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"5bc380f82bd19826cca2dfbab4481498be3ebeec","unresolved":false,"context_lines":[{"line_number":16,"context_line":"when a domain is disabled."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: If3e941018d5c2c9bd22397e69f83b7bf92643340"},{"line_number":19,"context_line":"Partial-Bug: 1532280"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1aa78d24_0b3bdd39","line":19,"in_reply_to":"1aa78d24_cb855508","updated":"2016-07-07 20:57:57.000000000","message":"It could close the bug - but the bug would still be present if caching is enabled. I\u0027m working on a few more patches to fix that case, too.","commit_id":"d53db1889e17d493202743246243936af90234b9"}],"keystone/tests/unit/test_v3_auth.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"36a1d5be6b47d28e1bf8a221565d5b4581e71307","unresolved":false,"context_lines":[{"line_number":2482,"context_line":"    # FIXME(lbragstad): Remove this test from this class and inherit the"},{"line_number":2483,"context_line":"    # version in TokenAPITest once bug 1532280 is fixed."},{"line_number":2484,"context_line":"    def test_trust_token_is_invalid_when_trustee_domain_disabled(self):"},{"line_number":2485,"context_line":"        # Remove this once revocation for domains is handled properly"},{"line_number":2486,"context_line":"        self.config_fixture.config("},{"line_number":2487,"context_line":"            group\u003d\u0027cache\u0027,"},{"line_number":2488,"context_line":"            enabled\u003dFalse)"},{"line_number":2489,"context_line":"        # create a new domain with new user in that domain"},{"line_number":2490,"context_line":"        new_domain_ref \u003d unit.new_domain_ref()"},{"line_number":2491,"context_line":"        self.resource_api.create_domain(new_domain_ref[\u0027id\u0027], new_domain_ref)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_086c8806","line":2488,"range":{"start_line":2485,"start_character":0,"end_line":2488,"end_character":26},"updated":"2016-07-07 18:53:15.000000000","message":"why is this only needed for fernet ?","commit_id":"382c79118193016f7dda7a96012bc598405adaea"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"568db9e82d7dcc19d188ef1897d2ea04e0f0c93c","unresolved":false,"context_lines":[{"line_number":2485,"context_line":"        # Remove this once revocation for domains is handled properly"},{"line_number":2486,"context_line":"        self.config_fixture.config("},{"line_number":2487,"context_line":"            group\u003d\u0027cache\u0027,"},{"line_number":2488,"context_line":"            enabled\u003dFalse)"},{"line_number":2489,"context_line":"        # create a new domain with new user in that domain"},{"line_number":2490,"context_line":"        new_domain_ref \u003d unit.new_domain_ref()"},{"line_number":2491,"context_line":"        self.resource_api.create_domain(new_domain_ref[\u0027id\u0027], new_domain_ref)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_06fd996d","line":2488,"in_reply_to":"1aa78d24_086c8806","updated":"2016-07-07 19:37:17.000000000","message":"Because a callback is registered that will go through and delete uuid tokens when a domain is disabled. This invalidates the cache when the uuid provider is used.\n\nhttps://github.com/openstack/keystone/blob/5f7377f5abaf46e491d9c580ec59547cd8478aa3/keystone/token/provider.py#L155-L158\nhttps://github.com/openstack/keystone/blob/5f7377f5abaf46e491d9c580ec59547cd8478aa3/keystone/token/provider.py#L474-L478\n\nThose methods should be made more generic for fernet.","commit_id":"382c79118193016f7dda7a96012bc598405adaea"}],"keystone/token/providers/common.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"36a1d5be6b47d28e1bf8a221565d5b4581e71307","unresolved":false,"context_lines":[{"line_number":358,"context_line":"                self.resource_api.assert_domain_enabled("},{"line_number":359,"context_line":"                    trustor_user_ref[\u0027domain_id\u0027])"},{"line_number":360,"context_line":"            except AssertionError:"},{"line_number":361,"context_line":"                raise exception.NotFound(_(\u0027Trustor domain is disabled.\u0027))"},{"line_number":362,"context_line":"            try:"},{"line_number":363,"context_line":"                self.resource_api.assert_domain_enabled("},{"line_number":364,"context_line":"                    trustee_user_ref[\u0027domain_id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_a8c1f4e5","line":361,"range":{"start_line":361,"start_character":32,"end_line":361,"end_character":40},"updated":"2016-07-07 18:53:15.000000000","message":"why NotFound ? what was not found ? perhaps this should be exception.TokenNotFound ?","commit_id":"382c79118193016f7dda7a96012bc598405adaea"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"568db9e82d7dcc19d188ef1897d2ea04e0f0c93c","unresolved":false,"context_lines":[{"line_number":358,"context_line":"                self.resource_api.assert_domain_enabled("},{"line_number":359,"context_line":"                    trustor_user_ref[\u0027domain_id\u0027])"},{"line_number":360,"context_line":"            except AssertionError:"},{"line_number":361,"context_line":"                raise exception.NotFound(_(\u0027Trustor domain is disabled.\u0027))"},{"line_number":362,"context_line":"            try:"},{"line_number":363,"context_line":"                self.resource_api.assert_domain_enabled("},{"line_number":364,"context_line":"                    trustee_user_ref[\u0027domain_id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_c64c31a0","line":361,"in_reply_to":"1aa78d24_a8c1f4e5","updated":"2016-07-07 19:37:17.000000000","message":"Done","commit_id":"382c79118193016f7dda7a96012bc598405adaea"}]}
