Opened 8 years ago

Closed 8 years ago

#27696 closed Bug (fixed)

Email line length limiting does not work properly with Cyrillic strings

Reported by: Pavel Pokrovskiy Owned by: nobody
Component: Core (Mail) Version: 1.10
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Fix for #22561 does not work properly on Cyrillic strings.

Tried it out with following snippet:

https://gist.github.com/ppokrovsky/d06d0d9e3c8f55bf15984ecd22954683

with body set to test_body_lat , the has_long_lines flag in django.core.mail.message.SafeMIMEText() is set to True and therefore correctly applies 'quoted-printable' encoding, while when body is set to test_body_ru , it leaves has_long_lines as False therefore leaving default charset, which results in broken email body.

Change History (3)

comment:1 by Tim Graham, 8 years ago

Has patch: set
Summary: Fix for #22561 does not work properly with Cyrillic stringsEmail line length limiting does not work properly with Cyrillic strings
Triage Stage: UnreviewedAccepted

PR from Claude.

comment:2 by Tim Graham, 8 years ago

Triage Stage: AcceptedReady for checkin

comment:3 by Claude Paroz <claude@…>, 8 years ago

Resolution: fixed
Status: newclosed

In 93905339:

Fixed #27696 -- Measured email long lines on encoded content

Thanks Pavel Pokrovskiy for the report and Tim Graham for the review.

Note: See TracTickets for help on using tickets.
Back to Top