)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"46daf8ef6cf377d0910db6d9fc1252d9ec0739c5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Standardize token scoping workflow."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently scoping the federated tokens required separate authenticion"},{"line_number":10,"context_line":"method \u0027saml2\u0027 or \u0027mapped\u0027."},{"line_number":11,"context_line":"Thanks to patch: https://review.openstack.org/#/c/130593/ this workflow"},{"line_number":12,"context_line":"was standardized."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a961159_04831caa","line":9,"updated":"2015-01-10 05:39:04.000000000","message":"nit: authentication","commit_id":"e48f3f5931f7defe2e0ced57ec3d6bcaa9d91e2d"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"b1b83c90a5b7739b62f316d7d0a3f13290a7f123","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Standardize token scoping workflow."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently scoping the federated tokens requires separate authentication"},{"line_number":10,"context_line":"method \u0027saml2\u0027 or \u0027mapped\u0027."},{"line_number":11,"context_line":"Thanks to patch: https://review.openstack.org/#/c/130593/ this workflow"},{"line_number":12,"context_line":"was standardized."},{"line_number":13,"context_line":"This patch handles the client side of the standardization process."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da86d52c_80dca3e5","line":10,"updated":"2015-02-19 08:16:16.000000000","message":"NIT: maybe add extra line in between paragraphs for readability.","commit_id":"182f3a25e7723f63ef2a3b53b1d9cc3ea9145acc"}],"keystoneclient/contrib/auth/v3/federation.py":[{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"46daf8ef6cf377d0910db6d9fc1252d9ec0739c5","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    @property"},{"line_number":28,"context_line":"    def scoped_token_plugin(self):"},{"line_number":29,"context_line":"        \"\"\"Return class that should be used for scoping the token.\"\"\""},{"line_number":30,"context_line":"        return v3.Token"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    @property"},{"line_number":33,"context_line":"    def token_url(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_64b9407c","line":30,"updated":"2015-01-10 05:39:04.000000000","message":"Is this method still needed if it will always be v3.Token?","commit_id":"e48f3f5931f7defe2e0ced57ec3d6bcaa9d91e2d"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"22bc231217410ea491fee03ccd6f7dcb825dc0b7","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    @property"},{"line_number":28,"context_line":"    def scoped_token_plugin(self):"},{"line_number":29,"context_line":"        \"\"\"Return class that should be used for scoping the token.\"\"\""},{"line_number":30,"context_line":"        return v3.Token"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    @property"},{"line_number":33,"context_line":"    def token_url(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_76e4843c","line":30,"in_reply_to":"3a961159_64b9407c","updated":"2015-01-12 08:51:28.000000000","message":"Your suggestions is to directly use v3.Token in line ~79 ? \nI\u0027d leave it here as it is now, so people can change the module in the underlying patches, if they need for some reason.","commit_id":"e48f3f5931f7defe2e0ced57ec3d6bcaa9d91e2d"}]}