exceptions: make error code be implicit
Change-Id: I7e3d1875c0e853e7ebd537afde7a2acb87e6dd9f
This commit is contained in:
@@ -16,16 +16,20 @@ class ClientException(Exception):
|
||||
"""The base exception class for all exceptions this library raises."""
|
||||
message = 'Unknown Error'
|
||||
|
||||
def __init__(self, code, message=None, request_id=None,
|
||||
def __init__(self, message=None, request_id=None,
|
||||
url=None, method=None):
|
||||
self.code = code
|
||||
self.message = message or self.__class__.message
|
||||
self.request_id = request_id
|
||||
self.url = url
|
||||
self.method = method
|
||||
|
||||
# NOTE(jd) for backward compat
|
||||
@property
|
||||
def code(self):
|
||||
return self.http_status
|
||||
|
||||
def __str__(self):
|
||||
formatted_string = "%s (HTTP %s)" % (self.message, self.code)
|
||||
formatted_string = "%s (HTTP %s)" % (self.message, self.http_status)
|
||||
if self.request_id:
|
||||
formatted_string += " (Request-ID: %s)" % self.request_id
|
||||
|
||||
@@ -154,7 +158,6 @@ def from_response(response, url, method=None):
|
||||
content_type = response.headers.get("Content-Type", "").split(";")[0]
|
||||
|
||||
kwargs = {
|
||||
'code': response.status_code,
|
||||
'method': method,
|
||||
'url': url,
|
||||
'request_id': req_id,
|
||||
|
||||
@@ -98,7 +98,7 @@ def _find_alarm_by_name(client, name, return_id=False):
|
||||
raise exceptions.NoUniqueMatch(msg)
|
||||
elif not alarms:
|
||||
msg = (_("Alarm %s not found") % name)
|
||||
raise exceptions.NotFound(404, msg)
|
||||
raise exceptions.NotFound(msg)
|
||||
else:
|
||||
if return_id:
|
||||
return alarms[0]['alarm_id']
|
||||
|
||||
Reference in New Issue
Block a user