)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"d2a81bf5c453aa98698a1751c4f10d3aff85acbb","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2015-01-27 11:48:20 +0800"},{"line_number":4,"context_line":"Commit:     wanghong \u003cw.wanghong@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-01-27 11:48:20 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add missing classmethod decorater for get_auth_context"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Authough it is not requred, but we use self for member method and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da86d52c_60ba0673","line":6,"updated":"2015-02-13 16:23:42.000000000","message":"decorator","commit_id":"d5db279eda350b1c894e6cd3d2b28abe4d121783"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"d2a81bf5c453aa98698a1751c4f10d3aff85acbb","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add missing classmethod decorater for get_auth_context"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Authough it is not requred, but we use self for member method and"},{"line_number":10,"context_line":"cls for class method generally."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I774460a9e802e873c93e0eb07ab454af0d4dd4df"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da86d52c_8098da1c","line":9,"updated":"2015-02-13 16:23:42.000000000","message":"required*","commit_id":"d5db279eda350b1c894e6cd3d2b28abe4d121783"}],"keystone/common/controller.py":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"f97ce696a480415176a219c0a8f6936f3777a275","unresolved":false,"context_lines":[{"line_number":318,"context_line":""},{"line_number":319,"context_line":"        return \u0027%s/%s/%s\u0027 % (endpoint, \u0027v3\u0027, path.lstrip(\u0027/\u0027))"},{"line_number":320,"context_line":""},{"line_number":321,"context_line":"    @classmethod"},{"line_number":322,"context_line":"    def get_auth_context(cls, context):"},{"line_number":323,"context_line":"        # TODO(dolphm): this method of accessing the auth context is terrible,"},{"line_number":324,"context_line":"        # but context needs to be refactored to always have reasonable values."}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_fa5691cd","line":321,"updated":"2015-02-14 03:18:08.000000000","message":"Is there any reason you made this a classmethod instead of just renaming cls to self? Nothing uses this as a classmethod at this point.","commit_id":"d5db279eda350b1c894e6cd3d2b28abe4d121783"},{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"af10b71e9666ff2d48817949ecc7fa519ae1eee4","unresolved":false,"context_lines":[{"line_number":353,"context_line":""},{"line_number":354,"context_line":"        return \u0027%s/%s/%s\u0027 % (endpoint, \u0027v3\u0027, path.lstrip(\u0027/\u0027))"},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"    def get_auth_context(self, context):"},{"line_number":357,"context_line":"        # TODO(dolphm): this method of accessing the auth context is terrible,"},{"line_number":358,"context_line":"        # but context needs to be refactored to always have reasonable values."},{"line_number":359,"context_line":"        env_context \u003d context.get(\u0027environment\u0027, {})"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_27f7df8d","line":356,"updated":"2015-02-17 16:18:07.000000000","message":"Why not to make it a @staticmethod?","commit_id":"0401f56964cf28754d76a52b26f6b1eb117c8972"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"2be2c5f9325837fdc90888a9d66b01e01341b5cd","unresolved":false,"context_lines":[{"line_number":353,"context_line":""},{"line_number":354,"context_line":"        return \u0027%s/%s/%s\u0027 % (endpoint, \u0027v3\u0027, path.lstrip(\u0027/\u0027))"},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"    def get_auth_context(self, context):"},{"line_number":357,"context_line":"        # TODO(dolphm): this method of accessing the auth context is terrible,"},{"line_number":358,"context_line":"        # but context needs to be refactored to always have reasonable values."},{"line_number":359,"context_line":"        env_context \u003d context.get(\u0027environment\u0027, {})"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_2e8eafd0","line":356,"in_reply_to":"da86d52c_27f7df8d","updated":"2015-02-25 03:53:27.000000000","message":"Currently, nothing uses this as a classmethod and I don\u0027t see the use case that we need use this as a classmethod.","commit_id":"0401f56964cf28754d76a52b26f6b1eb117c8972"}]}