)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"2ffa3ff8a7759fed1c3975b453afb9d59a72a1a2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     wanghong \u003cw.wanghong@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-02-03 17:48:00 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix another normal user can get other user\u0027s ec2 credential"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, policy.json and policy.v3cloudsample.json only check if"},{"line_number":10,"context_line":"the user from token matchs the user from url. However, we should"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da86d52c_abea1faa","line":7,"updated":"2015-02-13 16:34:10.000000000","message":"nit: \n\n    Make credential policy check ownership of credential","commit_id":"ce7f030fb5973a323c94a2213bfe734bc536a01e"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"26b7fd30d83fcfe239e537798bd2d435e6021fa1","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I5c8bbb6736b028d6cb693d2a35e018f28caeaa57"},{"line_number":14,"context_line":"Closes-Bug: #1417366"},{"line_number":15,"context_line":"Related-Bug: #1417522"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a80dd14_9624b20f","line":15,"updated":"2015-03-10 15:33:48.000000000","message":"Doesn\u0027t the patch close the bug?","commit_id":"24dfa27e7f610857b018c31ba8fff78ab967615e"}],"keystone/tests/test_v3_protection.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"2ffa3ff8a7759fed1c3975b453afb9d59a72a1a2","unresolved":false,"context_lines":[{"line_number":699,"context_line":"        another_user \u003d self.new_user_ref(domain_id\u003dself.domainA[\u0027id\u0027])"},{"line_number":700,"context_line":"        password \u003d another_user[\u0027password\u0027]"},{"line_number":701,"context_line":"        another_user \u003d self.identity_api.create_user(another_user)"},{"line_number":702,"context_line":""},{"line_number":703,"context_line":"        # Here we use ec2 API rather than credenti_api to create ec2"},{"line_number":704,"context_line":"        # credential, this is because we don\u0027t want to do a lot things that"},{"line_number":705,"context_line":"        # API already done"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_4b5c43f1","line":702,"updated":"2015-02-13 16:34:10.000000000","message":"credential_api","commit_id":"ce7f030fb5973a323c94a2213bfe734bc536a01e"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"3ce89641b87baafe2858bde5d3bf027a4aa51e7f","unresolved":false,"context_lines":[{"line_number":702,"context_line":""},{"line_number":703,"context_line":"        # Here we use ec2 API rather than credenti_api to create ec2"},{"line_number":704,"context_line":"        # credential, this is because we don\u0027t want to do a lot things that"},{"line_number":705,"context_line":"        # API already done"},{"line_number":706,"context_line":"        just_user_auth \u003d self.build_authentication_request("},{"line_number":707,"context_line":"            user_id\u003dself.just_a_user[\u0027id\u0027],"},{"line_number":708,"context_line":"            password\u003dself.just_a_user[\u0027password\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_459a676f","line":705,"updated":"2015-02-14 03:19:08.000000000","message":"Instead of this comment block here I would personally find it more useful with a docstring like: \"Tests getting ec2 credentials through the ec2 API.\"","commit_id":"ce7f030fb5973a323c94a2213bfe734bc536a01e"}]}