)]}'
{"novaclient/client.py":[{"author":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"change_message_id":"cedf0af6eca0e4b465b8e68c93c4a2a4710e7ff5","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        #   check only nova-related calls"},{"line_number":85,"context_line":"        # api_versions.check_headers(resp, self.api_version)"},{"line_number":86,"context_line":"        if raise_exc and resp.status_code \u003e\u003d 400:"},{"line_number":87,"context_line":"            exc \u003d exceptions.from_response(resp, body, url, method)"},{"line_number":88,"context_line":"            msg \u003d str(exc).split(\u0027(\u0027)"},{"line_number":89,"context_line":"            del(msg[1])"},{"line_number":90,"context_line":"            msg \u003d \u0027(\u0027.join(msg)"},{"line_number":91,"context_line":"            raise exceptions.CommandError(msg)"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        return resp, body"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5201f7_2c187784","line":91,"range":{"start_line":87,"start_character":12,"end_line":91,"end_character":46},"updated":"2016-12-30 18:35:11.000000000","message":"The code that\u0027s adding the HTTP error code to the message is in exceptions.py in python-novaclient:\n\n- https://github.com/openstack/python-novaclient/blob/084b0abb755ac36415058f485fc6b0f318d9c041/novaclient/exceptions.py#L132\n\nIt feels strange to then go and remove it here in client.py.\n\nAre only the flavor commands leaking HTTP error code info at the command line? Or are all commands leaking HTTP error info at the command line? \n\nEither way, as a rule you\u0027ll need a unit test demonstrating the bug before the fix will be accepted.","commit_id":"c3ca40a616e5d31a9635d3929110e7c3768c76e8"}]}
