)]}'
{"keystone/tests/unit/test_v3_federation.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"44ae8cecd571bc2b5391cb0928a74f7f7fe96c7c","unresolved":false,"context_lines":[{"line_number":2251,"context_line":"        \"\"\""},{"line_number":2252,"context_line":""},{"line_number":2253,"context_line":"        resp \u003d self._issue_unscoped_token()"},{"line_number":2254,"context_line":"        employee_unscoped_token_id \u003d resp.headers.get(\u0027X-Subject-Token\u0027)"},{"line_number":2255,"context_line":"        resp \u003d self.get(\u0027/OS-FEDERATION/projects\u0027,"},{"line_number":2256,"context_line":"                        token\u003demployee_unscoped_token_id)"},{"line_number":2257,"context_line":"        projects \u003d resp.result[\u0027projects\u0027]"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_89e4beac","line":2254,"updated":"2015-03-09 21:55:06.000000000","message":"Validation of this token is failing for some reason that *isn\u0027t* the formatter. I\u0027ve confirmed the token_data is built properly from the v3_token_data_helper, etc... \n\nI\u0027m still digging into this but wanted to push what I had since it\u0027s easier to maintain the rebases.","commit_id":"30ff3b49eea534cf28884a859deaf35d9f6de230"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5654afb381e2959ef70b80b2883827a6b28d51c0","unresolved":false,"context_lines":[{"line_number":2208,"context_line":"                                fernet.KeyRepositoryTestMixin):"},{"line_number":2209,"context_line":"    AUTH_METHOD \u003d \u0027token\u0027"},{"line_number":2210,"context_line":""},{"line_number":2211,"context_line":"    def setUp(self):"},{"line_number":2212,"context_line":"        super(FernetFederatedTokenTests, self).setUp()"},{"line_number":2213,"context_line":""},{"line_number":2214,"context_line":"    def load_fixtures(self, fixtures):"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_d5578900","line":2211,"updated":"2015-03-12 18:07:16.000000000","message":"(nit) is this needed?","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"c877c39b6a2693c8cd1ffa3c15cbeace938ca023","unresolved":false,"context_lines":[{"line_number":2208,"context_line":"                                fernet.KeyRepositoryTestMixin):"},{"line_number":2209,"context_line":"    AUTH_METHOD \u003d \u0027token\u0027"},{"line_number":2210,"context_line":""},{"line_number":2211,"context_line":"    def setUp(self):"},{"line_number":2212,"context_line":"        super(FernetFederatedTokenTests, self).setUp()"},{"line_number":2213,"context_line":""},{"line_number":2214,"context_line":"    def load_fixtures(self, fixtures):"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_205fc042","line":2211,"in_reply_to":"9a80dd14_d5578900","updated":"2015-03-12 19:53:23.000000000","message":"Nope","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5654afb381e2959ef70b80b2883827a6b28d51c0","unresolved":false,"context_lines":[{"line_number":2231,"context_line":"    def test_federated_unscoped_token(self):"},{"line_number":2232,"context_line":"        resp \u003d self._issue_unscoped_token()"},{"line_number":2233,"context_line":"        self.assertEqual(184, len(resp.headers[\u0027X-Subject-Token\u0027]))"},{"line_number":2234,"context_line":"        self.assertIsNotNone(resp.headers.get(\u0027X-Subject-Token\u0027))"},{"line_number":2235,"context_line":""},{"line_number":2236,"context_line":"    def test_federated_unscoped_token_with_multiple_groups(self):"},{"line_number":2237,"context_line":"        assertion \u003d \u0027ANOTHER_CUSTOMER_ASSERTION\u0027"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_50772702","line":2234,"updated":"2015-03-12 18:07:16.000000000","message":"Can it be None it the previous assert succeeds?","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"156dba6704eb67c11ac2505053ccab14617f5995","unresolved":false,"context_lines":[{"line_number":2231,"context_line":"    def test_federated_unscoped_token(self):"},{"line_number":2232,"context_line":"        resp \u003d self._issue_unscoped_token()"},{"line_number":2233,"context_line":"        self.assertEqual(184, len(resp.headers[\u0027X-Subject-Token\u0027]))"},{"line_number":2234,"context_line":"        self.assertIsNotNone(resp.headers.get(\u0027X-Subject-Token\u0027))"},{"line_number":2235,"context_line":""},{"line_number":2236,"context_line":"    def test_federated_unscoped_token_with_multiple_groups(self):"},{"line_number":2237,"context_line":"        assertion \u003d \u0027ANOTHER_CUSTOMER_ASSERTION\u0027"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_cb3b6ae9","line":2234,"in_reply_to":"9a80dd14_50772702","updated":"2015-03-12 18:28:34.000000000","message":"nope, I can remove the last assertion.","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5654afb381e2959ef70b80b2883827a6b28d51c0","unresolved":false,"context_lines":[{"line_number":2241,"context_line":"    def test_validate_federated_unscoped_token(self):"},{"line_number":2242,"context_line":"        resp \u003d self._issue_unscoped_token()"},{"line_number":2243,"context_line":"        unscoped_token \u003d resp.headers.get(\u0027X-Subject-Token\u0027)"},{"line_number":2244,"context_line":"        self.get(\u0027/auth/tokens/\u0027, headers\u003d{\u0027X-Subject-Token\u0027: unscoped_token})"},{"line_number":2245,"context_line":""},{"line_number":2246,"context_line":"    def test_fernet_full_workflow(self):"},{"line_number":2247,"context_line":"        \"\"\"Test \u0027standard\u0027 workflow for granting Fernet access tokens."}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_504e47c2","line":2244,"updated":"2015-03-12 18:07:16.000000000","message":"Is there anything else to assert here or are you just checking that it\u0027s valid?","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"156dba6704eb67c11ac2505053ccab14617f5995","unresolved":false,"context_lines":[{"line_number":2241,"context_line":"    def test_validate_federated_unscoped_token(self):"},{"line_number":2242,"context_line":"        resp \u003d self._issue_unscoped_token()"},{"line_number":2243,"context_line":"        unscoped_token \u003d resp.headers.get(\u0027X-Subject-Token\u0027)"},{"line_number":2244,"context_line":"        self.get(\u0027/auth/tokens/\u0027, headers\u003d{\u0027X-Subject-Token\u0027: unscoped_token})"},{"line_number":2245,"context_line":""},{"line_number":2246,"context_line":"    def test_fernet_full_workflow(self):"},{"line_number":2247,"context_line":"        \"\"\"Test \u0027standard\u0027 workflow for granting Fernet access tokens."}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_6b019e05","line":2244,"in_reply_to":"9a80dd14_504e47c2","updated":"2015-03-12 18:28:34.000000000","message":"just checking that the token is valid. I can add a comment.","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"}],"keystone/tests/unit/token/test_fernet_provider.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"a5d4648ce87eb8e2d951b0c461b03eaa867360d0","unresolved":false,"context_lines":[{"line_number":24,"context_line":"from keystone.token.providers.fernet import utils"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"CONF \u003d cfg.CONF"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"CONF \u003d config.CONF"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"9a80dd14_8e25695e","line":27,"updated":"2015-03-10 21:42:54.000000000","message":"another rebase issue? The following line should be removed, I think.","commit_id":"4597432ff2cb6115ef9837a17e2f871e7fb6eb07"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"9dc077dca64843b4637a9b9bdb3bdca33a8c921c","unresolved":false,"context_lines":[{"line_number":24,"context_line":"from keystone.token.providers.fernet import utils"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"CONF \u003d cfg.CONF"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"CONF \u003d config.CONF"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"9a80dd14_8f5aa74e","line":27,"in_reply_to":"9a80dd14_8e25695e","updated":"2015-03-11 13:30:20.000000000","message":"++","commit_id":"4597432ff2cb6115ef9837a17e2f871e7fb6eb07"}],"keystone/token/providers/fernet/core.py":[{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"f3b6ca4552d2093eef42642bd9a2092b8e95d94a","unresolved":false,"context_lines":[{"line_number":14,"context_line":"from oslo_log import log"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from keystone.common import dependency"},{"line_number":17,"context_line":"from keystone.contrib import federation as fed"},{"line_number":18,"context_line":"from keystone import exception"},{"line_number":19,"context_line":"from keystone.token.providers import common"},{"line_number":20,"context_line":"from keystone.token.providers.fernet import token_formatters as tf"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a80dd14_afcc066b","line":17,"updated":"2015-03-10 12:38:21.000000000","message":"uh, we use module \u0027federation\u0027 almost everywhere, so maybe we could stay with that?","commit_id":"11b736e8b34d184e864bbf89b4fa9c40bc654f8a"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"d922250ad62aece7085242b2a8ae5fa913671554","unresolved":false,"context_lines":[{"line_number":14,"context_line":"from oslo_log import log"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from keystone.common import dependency"},{"line_number":17,"context_line":"from keystone.contrib import federation as fed"},{"line_number":18,"context_line":"from keystone import exception"},{"line_number":19,"context_line":"from keystone.token.providers import common"},{"line_number":20,"context_line":"from keystone.token.providers.fernet import token_formatters as tf"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a80dd14_b0c9be7b","line":17,"in_reply_to":"9a80dd14_afcc066b","updated":"2015-03-10 17:52:01.000000000","message":"Done","commit_id":"11b736e8b34d184e864bbf89b4fa9c40bc654f8a"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"f3b6ca4552d2093eef42642bd9a2092b8e95d94a","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        methods \u003d [\u0027password\u0027, \u0027token\u0027]"},{"line_number":137,"context_line":"        if group_ids:"},{"line_number":138,"context_line":"            token_dict[\u0027user\u0027][fed.FEDERATION][\u0027groups\u0027] \u003d group_ids"},{"line_number":139,"context_line":"            methods \u003d [\u0027saml2\u0027]"},{"line_number":140,"context_line":"        trust_ref \u003d self.trust_api.get_trust(trust_id)"},{"line_number":141,"context_line":"        return self.v3_token_data_helper.get_token_data("},{"line_number":142,"context_line":"            user_id,"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a80dd14_8f37424c","line":139,"updated":"2015-03-10 12:38:21.000000000","message":"We cannot hardcode \u0027saml2\u0027. \nA rule of thumb is that:\n\n- for unscoped tokens auth method is equal to protocol used\n- for scoped tokens we use auth method \u0027token\u0027","commit_id":"11b736e8b34d184e864bbf89b4fa9c40bc654f8a"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"d922250ad62aece7085242b2a8ae5fa913671554","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        methods \u003d [\u0027password\u0027, \u0027token\u0027]"},{"line_number":137,"context_line":"        if group_ids:"},{"line_number":138,"context_line":"            token_dict[\u0027user\u0027][fed.FEDERATION][\u0027groups\u0027] \u003d group_ids"},{"line_number":139,"context_line":"            methods \u003d [\u0027saml2\u0027]"},{"line_number":140,"context_line":"        trust_ref \u003d self.trust_api.get_trust(trust_id)"},{"line_number":141,"context_line":"        return self.v3_token_data_helper.get_token_data("},{"line_number":142,"context_line":"            user_id,"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a80dd14_30930e51","line":139,"in_reply_to":"9a80dd14_8f37424c","updated":"2015-03-10 17:52:01.000000000","message":"++ agreed, this was experimental.","commit_id":"11b736e8b34d184e864bbf89b4fa9c40bc654f8a"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5654afb381e2959ef70b80b2883827a6b28d51c0","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        group_ids \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":57,"context_line":"            federation.FEDERATION, {}).get(\u0027groups\u0027)"},{"line_number":58,"context_line":"        idp_id \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":59,"context_line":"            federation.FEDERATION, {}).get(\u0027identity_provider\u0027).get(\u0027id\u0027)"},{"line_number":60,"context_line":"        protocol_id \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":61,"context_line":"            federation.FEDERATION, {}).get(\u0027protocol\u0027).get(\u0027id\u0027)"},{"line_number":62,"context_line":"        if not group_ids:"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_f0b9fb79","line":59,"updated":"2015-03-12 18:07:16.000000000","message":"If \u0027federation.FEDERATION\u0027 doesn\u0027t exist then the \u0027.get(\u0027identity_provider\u0027)\u0027 will return None and break on the \u0027.get(\u0027id\u0027)\u0027","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"156dba6704eb67c11ac2505053ccab14617f5995","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        group_ids \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":57,"context_line":"            federation.FEDERATION, {}).get(\u0027groups\u0027)"},{"line_number":58,"context_line":"        idp_id \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":59,"context_line":"            federation.FEDERATION, {}).get(\u0027identity_provider\u0027).get(\u0027id\u0027)"},{"line_number":60,"context_line":"        protocol_id \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":61,"context_line":"            federation.FEDERATION, {}).get(\u0027protocol\u0027).get(\u0027id\u0027)"},{"line_number":62,"context_line":"        if not group_ids:"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_7717c045","line":59,"in_reply_to":"9a80dd14_f0b9fb79","updated":"2015-03-12 18:28:34.000000000","message":"Yes, this should be\n\n    .get(\u0027identity_provider\u0027, {}).get(\u0027id\u0027)","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5654afb381e2959ef70b80b2883827a6b28d51c0","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        idp_id \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":59,"context_line":"            federation.FEDERATION, {}).get(\u0027identity_provider\u0027).get(\u0027id\u0027)"},{"line_number":60,"context_line":"        protocol_id \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":61,"context_line":"            federation.FEDERATION, {}).get(\u0027protocol\u0027).get(\u0027id\u0027)"},{"line_number":62,"context_line":"        if not group_ids:"},{"line_number":63,"context_line":"            group_ids \u003d list()"},{"line_number":64,"context_line":"        federated_dict \u003d dict(group_ids\u003dgroup_ids, idp_id\u003didp_id,"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_10b04f8a","line":61,"updated":"2015-03-12 18:07:16.000000000","message":"Same issue here","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"156dba6704eb67c11ac2505053ccab14617f5995","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        idp_id \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":59,"context_line":"            federation.FEDERATION, {}).get(\u0027identity_provider\u0027).get(\u0027id\u0027)"},{"line_number":60,"context_line":"        protocol_id \u003d token_data.get(\u0027user\u0027, {}).get("},{"line_number":61,"context_line":"            federation.FEDERATION, {}).get(\u0027protocol\u0027).get(\u0027id\u0027)"},{"line_number":62,"context_line":"        if not group_ids:"},{"line_number":63,"context_line":"            group_ids \u003d list()"},{"line_number":64,"context_line":"        federated_dict \u003d dict(group_ids\u003dgroup_ids, idp_id\u003didp_id,"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_f722d0a1","line":61,"in_reply_to":"9a80dd14_10b04f8a","updated":"2015-03-12 18:28:34.000000000","message":"I\u0027ll fix to be the same as above.","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5654afb381e2959ef70b80b2883827a6b28d51c0","unresolved":false,"context_lines":[{"line_number":141,"context_line":"            domain_id\u003ddomain_id,"},{"line_number":142,"context_line":"            project_id\u003dproject_id,"},{"line_number":143,"context_line":"            trust_id\u003dtoken_data[\u0027token\u0027].get(\u0027OS-TRUST:trust\u0027, {}).get(\u0027id\u0027),"},{"line_number":144,"context_line":"            federated_info\u003dfederated_dict if federated_dict else None)"},{"line_number":145,"context_line":"        return token, token_data"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"    def validate_v2_token(self, token_ref):"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_702a6b91","line":144,"updated":"2015-03-12 18:07:16.000000000","message":"isn\u0027t federated_dict alreayd either a dict or None?","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"156dba6704eb67c11ac2505053ccab14617f5995","unresolved":false,"context_lines":[{"line_number":141,"context_line":"            domain_id\u003ddomain_id,"},{"line_number":142,"context_line":"            project_id\u003dproject_id,"},{"line_number":143,"context_line":"            trust_id\u003dtoken_data[\u0027token\u0027].get(\u0027OS-TRUST:trust\u0027, {}).get(\u0027id\u0027),"},{"line_number":144,"context_line":"            federated_info\u003dfederated_dict if federated_dict else None)"},{"line_number":145,"context_line":"        return token, token_data"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"    def validate_v2_token(self, token_ref):"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a80dd14_772ee08d","line":144,"in_reply_to":"9a80dd14_702a6b91","updated":"2015-03-12 18:28:34.000000000","message":"++ yes, you\u0027re right.","commit_id":"13793d1d54db469c9fd7577eb8367d7a0f6112fb"}],"keystone/token/providers/fernet/token_formatters.py":[{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"f3b6ca4552d2093eef42642bd9a2092b8e95d94a","unresolved":false,"context_lines":[{"line_number":100,"context_line":"                project_id,"},{"line_number":101,"context_line":"                audit_ids,"},{"line_number":102,"context_line":"                trust_id)"},{"line_number":103,"context_line":"        elif group_ids:"},{"line_number":104,"context_line":"            version \u003d FederatedPayload.version"},{"line_number":105,"context_line":"            payload \u003d FederatedPayload.assemble("},{"line_number":106,"context_line":"                user_id,"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a80dd14_af56c659","line":103,"updated":"2015-03-10 12:38:21.000000000","message":"in federated tokens it\u0027s quite important to also have identity provider id and the protocol id.","commit_id":"11b736e8b34d184e864bbf89b4fa9c40bc654f8a"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"d109308cf247540481f223b6c9d7e025fccc84f3","unresolved":false,"context_lines":[{"line_number":100,"context_line":"                project_id,"},{"line_number":101,"context_line":"                audit_ids,"},{"line_number":102,"context_line":"                trust_id)"},{"line_number":103,"context_line":"        elif group_ids:"},{"line_number":104,"context_line":"            version \u003d FederatedPayload.version"},{"line_number":105,"context_line":"            payload \u003d FederatedPayload.assemble("},{"line_number":106,"context_line":"                user_id,"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a80dd14_7f98ea6a","line":103,"in_reply_to":"9a80dd14_af56c659","updated":"2015-03-10 17:52:19.000000000","message":"passing those in as well.","commit_id":"11b736e8b34d184e864bbf89b4fa9c40bc654f8a"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"bfeab596f027a0a92495f52db937f8792cce1957","unresolved":false,"context_lines":[{"line_number":435,"context_line":"        b_audit_ids \u003d map(provider.random_urlsafe_str_to_bytes, audit_ids)"},{"line_number":436,"context_line":""},{"line_number":437,"context_line":"        return (b_user_id, b_group_ids, b_idp_id, protocol_id,"},{"line_number":438,"context_line":"                b_federated_domain_id, federated_domain_name, b_audit_ids)"},{"line_number":439,"context_line":""},{"line_number":440,"context_line":"    @classmethod"},{"line_number":441,"context_line":"    def disassemble(cls, payload):"}],"source_content_type":"text/x-python","patch_set":10,"id":"9a80dd14_630f472f","line":438,"updated":"2015-03-11 15:51:19.000000000","message":"Not all of this is needed. It should be the following: \n\n    (b_user_id, b_group_ids, b_idp_id, protocol_id, b_audit_ids)\n\nThe federated domain id and the federated domain names are not needed since it is assumed if the token is a mapped token then the user is in the \"Federated\" domain.","commit_id":"4597432ff2cb6115ef9837a17e2f871e7fb6eb07"}]}