diff options
author | Tulio Casagrande <tcasagra@thoughtworks.com> | 2017-02-09 19:43:48 -0200 |
---|---|---|
committer | Tulio Casagrande <tcasagra@thoughtworks.com> | 2017-02-10 16:28:40 -0200 |
commit | cf7d2c516a897e6b19e0e11fd8c104e68a0951ad (patch) | |
tree | 2377d3d1923ddbbdb1d68d0d4d170a777c394490 /service/pixelated | |
parent | 2753ec89b925f5038a85d6bc1fad7b5c887dbf1e (diff) |
[#907] Remove unused login error handler
The handle_error_deferred method is supposed to return the error message
before the request is finished. This is not the case during the user
services setup, since the request was already finished and returned
with the interstitial page.
Diffstat (limited to 'service/pixelated')
-rw-r--r-- | service/pixelated/resources/login_resource.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/service/pixelated/resources/login_resource.py b/service/pixelated/resources/login_resource.py index 4d78174f..424e051b 100644 --- a/service/pixelated/resources/login_resource.py +++ b/service/pixelated/resources/login_resource.py @@ -20,7 +20,7 @@ from xml.sax import SAXParseException from pixelated.authentication import Authenticator from pixelated.config.leap import BootstrapUserServices from pixelated.resources import BaseResource, UnAuthorizedResource, IPixelatedSession -from pixelated.resources import handle_error_deferred, get_startup_folder +from pixelated.resources import get_startup_folder from twisted.cred.error import UnauthorizedLogin from twisted.internet import defer from twisted.logger import Logger @@ -139,8 +139,6 @@ class LoginResource(BaseResource): d = self._handle_login(request) d.addCallbacks(render_response, render_error) - d.addErrback(handle_error_deferred, request) - return NOT_DONE_YET @defer.inlineCallbacks @@ -151,7 +149,7 @@ class LoginResource(BaseResource): defer.returnValue(user_auth) def _complete_bootstrap(self, user_auth, request): - def log_error(error): + def login_error(error): log.error('Login error during %s services setup: %s \n %s' % (user_auth.username, error.getErrorMessage(), error.getTraceback())) def set_session_cookies(_): @@ -162,4 +160,4 @@ class LoginResource(BaseResource): password = request.args['password'][0] d = self._bootstrap_user_services.setup(user_auth, password, language) d.addCallback(set_session_cookies) - d.addErrback(log_error) + d.addErrback(login_error) |