)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ac83d6b5af946fadd7d915136110d00994fcfd9c","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Normally I try not to mix in cleanups, but whilst doing this and"},{"line_number":14,"context_line":"changing variables in list_trusts a small reshuffle and some whitespace"},{"line_number":15,"context_line":"just made the whole thing actually readable."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I5b64210fc797961b422a0ab9a1b4cee078fe6a0f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1aa78d24_3d3a17e8","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":43},"updated":"2016-07-14 21:30:13.000000000","message":"true","commit_id":"b3e065efaaaecf3a2fe83f45631567d63e19f47b"}],"keystone/trust/controllers.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"f8bda7543b6170070940e48e4d419616be5df963","unresolved":false,"context_lines":[{"line_number":195,"context_line":"            trusts +\u003d self.trust_api.list_trusts()"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        if trustor_user_id:"},{"line_number":198,"context_line":"            if trustor_user_id !\u003d request.context.user_id:"},{"line_number":199,"context_line":"                raise exception.Forbidden()"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"            trusts +\u003d self.trust_api.list_trusts_for_trustor(trustor_user_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"fad55976_1eaddd31","line":198,"updated":"2016-07-15 15:26:46.000000000","message":"This seems like stuff that should be handled by the trust Manager().","commit_id":"b3e065efaaaecf3a2fe83f45631567d63e19f47b"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"c91c6fb38384be99331fb2b9b3459e6907ed7c14","unresolved":false,"context_lines":[{"line_number":195,"context_line":"            trusts +\u003d self.trust_api.list_trusts()"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        if trustor_user_id:"},{"line_number":198,"context_line":"            if trustor_user_id !\u003d request.context.user_id:"},{"line_number":199,"context_line":"                raise exception.Forbidden()"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"            trusts +\u003d self.trust_api.list_trusts_for_trustor(trustor_user_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_4789d325","line":198,"in_reply_to":"fad55976_1eaddd31","updated":"2016-07-19 23:30:05.000000000","message":"there is a lot of places i\u0027m discovering where the line between controller, manager and driver are badly blurred.","commit_id":"b3e065efaaaecf3a2fe83f45631567d63e19f47b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ac83d6b5af946fadd7d915136110d00994fcfd9c","unresolved":false,"context_lines":[{"line_number":194,"context_line":"            self.assert_admin(request)"},{"line_number":195,"context_line":"            trusts +\u003d self.trust_api.list_trusts()"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        if trustor_user_id:"},{"line_number":198,"context_line":"            if trustor_user_id !\u003d request.context.user_id:"},{"line_number":199,"context_line":"                raise exception.Forbidden()"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"            trusts +\u003d self.trust_api.list_trusts_for_trustor(trustor_user_id)"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"        if trustee_user_id:"},{"line_number":204,"context_line":"            if trustee_user_id !\u003d request.context.user_id:"},{"line_number":205,"context_line":"                raise exception.Forbidden()"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"            trusts +\u003d self.trust_api.list_trusts_for_trustee(trustee_user_id)"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"        for trust in trusts:"},{"line_number":210,"context_line":"            # get_trust returns roles, list_trusts does not"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_7d5dbf47","line":207,"range":{"start_line":197,"start_character":8,"end_line":207,"end_character":77},"updated":"2016-07-14 21:30:13.000000000","message":"this is so much more readable","commit_id":"b3e065efaaaecf3a2fe83f45631567d63e19f47b"}]}