)]}'
{"keystone/token/controllers.py":[{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"1fad2f06d12fbf487dd8267b7a771d9623c0d1db","unresolved":false,"context_lines":[{"line_number":193,"context_line":""},{"line_number":194,"context_line":"        metadata_ref \u003d {}"},{"line_number":195,"context_line":"        tenant_ref, metadata_ref[\u0027roles\u0027] \u003d self._get_project_roles_and_ref("},{"line_number":196,"context_line":"            user_id, tenant_id)"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        expiry \u003d token_model_ref.expires"},{"line_number":199,"context_line":"        if CONF.trust.enabled and \u0027trust_id\u0027 in auth:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a961159_f6c32d36","line":196,"updated":"2014-12-07 19:46:05.000000000","message":"We could put lines 194-196 just before line 186.\n\nAfter that, lines 197-215 could be placed on the method  _check_trust(..) you created. That method should then receive metadata_ref as argument and add \u0027roles\u0027, \u0027trustee_user_id\u0027 and \u0027trust_id\u0027  to it, if the preconditions are satisfied.\n\nNote that the verification at line 199 is the same condition we have after line 234.","commit_id":"42873bd804e3745e27056bca63d884b4f0f460d8"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"e04ceb078c9aa7e81bdfdadefab543ef60ded6c7","unresolved":false,"context_lines":[{"line_number":193,"context_line":""},{"line_number":194,"context_line":"        metadata_ref \u003d {}"},{"line_number":195,"context_line":"        tenant_ref, metadata_ref[\u0027roles\u0027] \u003d self._get_project_roles_and_ref("},{"line_number":196,"context_line":"            user_id, tenant_id)"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        expiry \u003d token_model_ref.expires"},{"line_number":199,"context_line":"        if CONF.trust.enabled and \u0027trust_id\u0027 in auth:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a961159_ecc36036","line":196,"in_reply_to":"3a961159_f6c32d36","updated":"2014-12-07 23:05:23.000000000","message":"I\u0027m not going to make the suggested change since it\u0027s unrelated to the change that\u0027s proposed, which is just to move some code around. If you want something else done then submit the change.","commit_id":"42873bd804e3745e27056bca63d884b4f0f460d8"}]}