diff --git a/apigateway/utils.py b/apigateway/utils.py index 83ada09..bf3b934 100644 --- a/apigateway/utils.py +++ b/apigateway/utils.py @@ -102,7 +102,7 @@ def send_feedback_email( recipient = current_app.config["FEEDBACK_EMAIL_SUBJECT_OVERRIDE"].get(subject, default_email) message = EmailMessage() - message["Subject"] = f"[{subject}] from {submitter_name} ({submitter_email})" + message["Subject"] = f"{subject} from {submitter_name} ({submitter_email})" message["From"] = f"ADS Administation <{default_email}>" message["To"] = recipient message["reply-to"] = f"{submitter_name} <{submitter_email}>" diff --git a/apigateway/views.py b/apigateway/views.py index 4d6341d..70a9c89 100644 --- a/apigateway/views.py +++ b/apigateway/views.py @@ -727,8 +727,8 @@ def _send_email(self, params, submitter_email, subject, email_body, attachments) attachments, ) return True - except: # noqa - current_app.logger.exception("Failed to send feedback email from user: {}".format(submitter_email)) + except Exception as e: # noqa + current_app.logger.error("Failed to send feedback email from user: {} with error {}".format(submitter_email, e)) return False def _post_to_slack(self, slack_data):