)]}'
{"keystone/token/provider.py":[{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"0eb13925d7f9ab599a23fd9db521e2b7620876ab","unresolved":false,"context_lines":[{"line_number":209,"context_line":"        else:"},{"line_number":210,"context_line":"            token_ref \u003d token_id"},{"line_number":211,"context_line":"        token \u003d self._validate_v2_token(token_ref)"},{"line_number":212,"context_line":"        token[\u0027access\u0027][\u0027token\u0027][\u0027id\u0027] \u003d token_id"},{"line_number":213,"context_line":"        self._token_belongs_to(token, belongs_to)"},{"line_number":214,"context_line":"        self._is_valid_token(token)"},{"line_number":215,"context_line":"        return token"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_7d8910ca","line":212,"updated":"2015-06-29 07:59:20.000000000","message":"So my understanding is that you simply don\u0027t want to validate id, as id\u0027s value is not a subject of validation?","commit_id":"486e480e5e47ae93d021212dc0a0df40c532bd44"}],"keystone/token/providers/common.py":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"5b417c5bde69394341a473a2d0409360708ef9c9","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        role_ids \u003d []"},{"line_number":91,"context_line":"        for role in v3_token.get(\u0027roles\u0027, []):"},{"line_number":92,"context_line":"            # Filter role id since it\u0027s not included in v2 token response"},{"line_number":93,"context_line":"            role_ids.append(role.pop(\u0027id\u0027))"},{"line_number":94,"context_line":"            user[\u0027roles\u0027].append(role)"},{"line_number":95,"context_line":"        user[\u0027roles_links\u0027] \u003d []"},{"line_number":96,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_103c96df","side":"PARENT","line":93,"updated":"2015-08-13 06:41:04.000000000","message":"why\u0027s this get taken out?","commit_id":"6982dc2fc20a80f00f01a37b8eb844c46ce1a752"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"08ceebb8468e4f047bb15a4654d29056d82083fc","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        role_ids \u003d []"},{"line_number":91,"context_line":"        for role in v3_token.get(\u0027roles\u0027, []):"},{"line_number":92,"context_line":"            # Filter role id since it\u0027s not included in v2 token response"},{"line_number":93,"context_line":"            role_ids.append(role.pop(\u0027id\u0027))"},{"line_number":94,"context_line":"            user[\u0027roles\u0027].append(role)"},{"line_number":95,"context_line":"        user[\u0027roles_links\u0027] \u003d []"},{"line_number":96,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"fa1b9901_79e7ca40","side":"PARENT","line":93,"in_reply_to":"1a4dcd0f_103c96df","updated":"2015-08-17 21:25:25.000000000","message":"This is relevant for the trust data sets iirc. we inconsistently used role_id in v2.","commit_id":"6982dc2fc20a80f00f01a37b8eb844c46ce1a752"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"53172d3090450408a63236dcc27c6c299a3b1b22","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                \u0027id\u0027: v3_trust_data[\u0027id\u0027],"},{"line_number":94,"context_line":"                \u0027trustor_user_id\u0027: v3_trust_data[\u0027trustor_user\u0027][\u0027id\u0027],"},{"line_number":95,"context_line":"                \u0027impersonation\u0027: v3_trust_data[\u0027impersonation\u0027]"},{"line_number":96,"context_line":"            }"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        # Set user roles"},{"line_number":99,"context_line":"        user[\u0027roles\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_7eee1b23","line":96,"updated":"2015-08-13 15:00:23.000000000","message":"Yeah, I was hitting this issue consolidating the v2 issue token path, I\u0027ll have to refactor the v2 token provider stuff once this lands.","commit_id":"885b13179fba5e22361a942f7ca33bd914bba703"}]}