From b1b2c0d0997ac29b775f001f632e3327ce86509b Mon Sep 17 00:00:00 2001 From: Rapptz Date: Fri, 27 Nov 2015 17:03:36 -0500 Subject: Add LoginFailure exception for a clearer failure in Client.login The older HTTPException is not exactly the clearest thing for people who are new to programming or HTTP exceptions in general. --- discord/client.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'discord/client.py') diff --git a/discord/client.py b/discord/client.py index e182347d..203664dc 100644 --- a/discord/client.py +++ b/discord/client.py @@ -810,7 +810,7 @@ class Client(object): After this function is called, :attr:`is_logged_in` returns True if no errors occur. If an error occurs during the login process, then - :exc:`HTTPException` is raised. + :exc:`LoginFailure` or :exc:`HTTPException` is raised. This function raises :exc:`GatewayNotFound` if it was unavailable to connect to a websocket gateway. @@ -826,7 +826,10 @@ class Client(object): r = requests.post(endpoints.LOGIN, json=payload) log.debug(request_logging_format.format(response=r)) - utils._verify_successful_response(r) + if r.status_code == 400: + raise LoginFailure('Improper credentials have been passed.') + elif r.status_code != 200: + raise HTTPException(r) log.info('logging in returned status code {}'.format(r.status_code)) self.email = email -- cgit v1.2.3