Skip to content

Commit 4afd1ae

Browse files
committed
Add exceptions for missing offline_token and access token error response
1 parent ddbf38c commit 4afd1ae

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

manifester/manifester.py

+6-2
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
import string
1010

1111
from dynaconf.utils.boxing import DynaBox
12-
from requests.exceptions import Timeout
12+
from requests.exceptions import RequestException, Timeout
1313

1414
from manifester.helpers import (
1515
fetch_paginated_data,
@@ -35,8 +35,10 @@ def __init__(
3535
if minimal_init:
3636
if kwargs.get("offline_token") is not None:
3737
self.offline_token = kwargs.get("offline_token")
38-
else:
38+
elif settings.get("offline_token") is not None:
3939
self.offline_token = settings.get("offline_token")
40+
else:
41+
raise KeyError("Offline token not defined.")
4042
self.token_request_url = settings.get("url").get("token_request")
4143
self.allocations_url = settings.get("url").get("allocations")
4244
self._access_token = None
@@ -113,6 +115,8 @@ def access_token(self):
113115
cmd_args=[f"{self.token_request_url}"],
114116
cmd_kwargs=token_request_data,
115117
).json()
118+
if 'error' in token_data:
119+
raise RequestException(f"{token_data["error"]}: {token_data["error_description"]}")
116120
if self.is_mock:
117121
self._access_token = token_data.access_token
118122
else:

0 commit comments

Comments
 (0)