Don't use person.plain_name() when constructing email addresses, as it can contain unicode characters which the current email pipeline can't handle correctly. Use person.plain_ascii() instead (or person.formatted_email()).

- Legacy-Id: 12865
2017-02-16 22:59:53 +00:00 · 2017-02-16 22:59:53 +00:00 · b8815b00c5
parent 3d56952d56
commit b8815b00c5
1 changed files with 1 additions and 3 deletions
--- a/ietf/doc/views_review.py
+++ b/ietf/doc/views_review.py
@ -559,9 +559,7 @@ def complete_review(request, name, request_id):
            if need_to_email_review:
                # email the review
                subject = "{} of {}-{}".format("Partial review" if review_req.state_id == "part-completed" else "Review", review_req.doc.name, review_req.reviewed_rev)
-                msg = send_mail(request, to, 
-                                (request.user.person.plain_name(),request.user.person.email_address()),
-                                subject,
+                msg = send_mail(request, to, request.user.person.formatted_email(), subject,
                                "review/completed_review.txt", {
                                    "review_req": review_req,
                                    "content": encoded_content.decode("utf-8"),