)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2861,"name":"Vincent Hou","email":"shou@us.ibm.com","username":"houshengbo"},"change_message_id":"f536563d7275e297daf1b9f8fe5b137734911797","unresolved":false,"context_lines":[{"line_number":10,"context_line":"2. Define \"body \u003d None\" before if-block to get rid of body-statement both in"},{"line_number":11,"context_line":"   except-block and else-case."},{"line_number":12,"context_line":"3. Acknowledge error occurred instead of silently squashing it."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I31e17827b40e7a15d88ecacf1fdb2abbbe2da5c0"},{"line_number":15,"context_line":"Signed-off-by: Lin Yang \u003clin.a.yang@intel.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba3cc151_7b61ad38","line":13,"updated":"2015-07-01 07:42:11.000000000","message":"Any bug or BP link?","commit_id":"c7792f6200f9d664ab2bf3713c76d0088d237d44"}],"cinderclient/client.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"d4671887cfabbc76399e44fb4b8d60185c029217","unresolved":false,"context_lines":[{"line_number":264,"context_line":"            try:"},{"line_number":265,"context_line":"                body \u003d json.loads(resp.text)"},{"line_number":266,"context_line":"            except ValueError as e:"},{"line_number":267,"context_line":"                self._logger.debug(\"Load http response text error: %s\" % e)"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        if resp.status_code \u003e\u003d 400:"},{"line_number":270,"context_line":"            raise exceptions.from_response(resp, body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_da49d7b8","line":267,"updated":"2015-07-07 18:46:52.000000000","message":"The formatting % should be changed to , so the logger can format the string only if and when it is needed as well as performing any unicode conversion on the ValueError exception. If e contains any unicode characters this string formatting could cause an exception.","commit_id":"c7792f6200f9d664ab2bf3713c76d0088d237d44"},{"author":{"_account_id":12597,"name":"Lin Yang","email":"lin.a.yang@intel.com","username":"lin.yang"},"change_message_id":"83ad802a946b816a067e51c00b7bb1c082f8042c","unresolved":false,"context_lines":[{"line_number":264,"context_line":"            try:"},{"line_number":265,"context_line":"                body \u003d json.loads(resp.text)"},{"line_number":266,"context_line":"            except ValueError as e:"},{"line_number":267,"context_line":"                self._logger.debug(\"Load http response text error: %s\" % e)"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        if resp.status_code \u003e\u003d 400:"},{"line_number":270,"context_line":"            raise exceptions.from_response(resp, body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_caf6d75c","line":267,"in_reply_to":"ba3cc151_da49d7b8","updated":"2015-07-08 03:39:36.000000000","message":"Thanks for pointing out this. Update it soon.","commit_id":"c7792f6200f9d664ab2bf3713c76d0088d237d44"}]}