From 5c7a321ba256e2d153fd7878385ea5dc1fcfaad1 Mon Sep 17 00:00:00 2001 From: Alexey Pelykh Date: Fri, 24 Apr 2020 08:09:21 +0200 Subject: [PATCH] [IMP] account_bank_statement_import_online: better logging --- .../models/online_bank_statement_provider.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/account_bank_statement_import_online/models/online_bank_statement_provider.py b/account_bank_statement_import_online/models/online_bank_statement_provider.py index ee1cbcb..2d44a0d 100644 --- a/account_bank_statement_import_online/models/online_bank_statement_provider.py +++ b/account_bank_statement_import_online/models/online_bank_statement_provider.py @@ -5,6 +5,7 @@ from datetime import datetime from dateutil.relativedelta import relativedelta, MO from decimal import Decimal +from html import escape import logging from pytz import timezone, utc from sys import exc_info @@ -182,7 +183,6 @@ class OnlineBankStatementProvider(models.Model): statement_date_until ) except: - e = exc_info()[1] if is_scheduled: _logger.warning( 'Online Bank Statement Provider "%s" failed to' @@ -195,16 +195,16 @@ class OnlineBankStatementProvider(models.Model): ) provider.message_post( body=_( - 'Online Bank Statement Provider "%s" failed to' - ' obtain statement data since %s until %s:\n%s' + 'Failed to obtain statement data for period ' + 'since %s until %s: %s. See server logs for ' + 'more details.' ) % ( - provider.name, statement_date_since, statement_date_until, - str(e) if e else _('N/A'), + escape(str(exc_info()[1])) or _('N/A') ), subject=_( - 'Online Bank Statement Provider failure' + 'Issue with Online Bank Statement Provider' ), ) break