diff options
author | Thais Siqueira <thais.siqueira@gmail.com> | 2017-03-31 17:32:49 -0300 |
---|---|---|
committer | Thais Siqueira <thais.siqueira@gmail.com> | 2017-03-31 17:32:49 -0300 |
commit | 304215fe36666c8401bd9fa01a50cab61ddc10ca (patch) | |
tree | 58d15414134ef5554e947348179e92e248df433b /service/pixelated | |
parent | da79ad444e3021c785c9b5d9c6cf8df8836d8563 (diff) |
[#927] Log exception with recovery code
with @tayanefernandes
Diffstat (limited to 'service/pixelated')
-rw-r--r-- | service/pixelated/account_recovery.py | 9 | ||||
-rw-r--r-- | service/pixelated/resources/__init__.py | 3 | ||||
-rw-r--r-- | service/pixelated/resources/backup_account_resource.py | 7 |
3 files changed, 11 insertions, 8 deletions
diff --git a/service/pixelated/account_recovery.py b/service/pixelated/account_recovery.py index fa1bf4f9..c0a18792 100644 --- a/service/pixelated/account_recovery.py +++ b/service/pixelated/account_recovery.py @@ -41,8 +41,9 @@ class AccountRecovery(object): returnValue(response) except Exception as e: - log.warn('Something went wrong when trying to save the recovery code') - raise + log.error('Something went wrong when trying to save the recovery code') + log.error(e) + raise e @inlineCallbacks def _send_mail(self, code, backup_email): @@ -59,5 +60,5 @@ class AccountRecovery(object): msg.as_string()) returnValue(send_mail_result) except Exception as e: - log.warn('Failed trying to send the email with the recovery code') - raise + log.error('Failed trying to send the email with the recovery code') + raise e diff --git a/service/pixelated/resources/__init__.py b/service/pixelated/resources/__init__.py index 3b2a7118..61e60720 100644 --- a/service/pixelated/resources/__init__.py +++ b/service/pixelated/resources/__init__.py @@ -122,9 +122,6 @@ class BaseResource(Resource): def soledad(self, request): return self._service(request, '_leap_session').soledad - def get_backup_email(self, request): - return json.loads(request.content.getvalue()).get('backupEmail') - class UnAuthorizedResource(Resource): diff --git a/service/pixelated/resources/backup_account_resource.py b/service/pixelated/resources/backup_account_resource.py index 677e9e75..f51ac2ec 100644 --- a/service/pixelated/resources/backup_account_resource.py +++ b/service/pixelated/resources/backup_account_resource.py @@ -15,6 +15,8 @@ # along with Pixelated. If not, see <http://www.gnu.org/licenses/>. import os +import json + from xml.sax import SAXParseException from pixelated.resources import BaseResource @@ -53,7 +55,7 @@ class BackupAccountResource(BaseResource): self._authenticator.bonafide_session, self.soledad(request), self._service(request, '_leap_session').smtp_config, - self.get_backup_email(request)) + self._get_backup_email(request)) def update_response(response): request.setResponseCode(NO_CONTENT) @@ -66,3 +68,6 @@ class BackupAccountResource(BaseResource): d = account_recovery.update_recovery_code() d.addCallbacks(update_response, error_response) return NOT_DONE_YET + + def _get_backup_email(self, request): + return json.loads(request.content.getvalue()).get('backupEmail') |