Skip to content

Commit f0f99c9

Browse files
authored
Merge pull request #7 from nbuchwitz/devel/state-possible-credential-error
fix: Hint for wrong credentials
2 parents 0e3cc09 + 49ce284 commit f0f99c9

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

nc_dnsapi/__init__.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,13 +91,18 @@ def request(self, action, **kwargs):
9191

9292
if data['status'] == 'success':
9393
return data
94-
# empty dns zone
9594
elif data['statuscode'] == 5029:
95+
# empty dns zone
9696
return []
9797
else:
98-
raise Exception("{} ({})".format(data['longmessage'], data['statuscode']))
98+
exception_message = f"{data['longmessage']} ({data['statuscode']})"
99+
if data['statuscode'] == 4013 and action == "login":
100+
exception_message += ". This error occured during login and is usually caused" \
101+
" by wrong API credentials. Please check the provided values."
102+
103+
raise Exception(exception_message)
99104
else:
100-
raise Exception("{} ({})".format(response.reason, response.status_code))
105+
raise Exception("f{response.reason} ({response.status_code})")
101106

102107
def logout(self):
103108
self.request("logout")

0 commit comments

Comments
 (0)