Skip to content

Commit e827afa

Browse files
committed
Return correct Exception, depending on status_code (404, 429). Add missing Raise, comments. Fixes #385
1 parent d35d582 commit e827afa

3 files changed

Lines changed: 3 additions & 2 deletions

File tree

tidalapi/album.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,7 @@ def __init__(self, session: "Session", album_id: Optional[str]):
8989
request = self.request.request("GET", "albums/%s" % self.id)
9090
except ObjectNotFound as e:
9191
e.args = ("Album with id %s not found" % self.id,)
92+
raise e
9293
except TooManyRequests as e:
9394
e.args = ("Album unavailable",)
9495
raise e

tidalapi/exceptions.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ def http_error_to_tidal_error(http_error: HTTPError) -> TidalAPIError | None:
7777
else:
7878
log.debug("Request response: '%s'", json.dumps(json_data))
7979

80-
elif response.status_code == 404:
80+
if response.status_code == 404:
8181
return ObjectNotFound("Object not found")
8282
elif response.status_code == 429:
8383
retry_after = int(response.headers.get("Retry-After", -1))

tidalapi/request.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ def request(
157157
if err := http_error_to_tidal_error(e):
158158
raise err from e
159159
else:
160-
raise # re raise last error, usually HTTPError
160+
raise # re-raise last error, usually HTTPError
161161
return request
162162

163163
def get_latest_err_response(self) -> dict:

0 commit comments

Comments
 (0)