summaryrefslogtreecommitdiffstats
path: root/Lib
diff options
context:
space:
mode:
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>2024-01-26 16:30:32 (GMT)
committerGitHub <noreply@github.com>2024-01-26 16:30:32 (GMT)
commitfd8aafd64d23e7de51566c7099c704d48da86eea (patch)
tree75640919672f0b3d44abee11e7c369e33d0174f4 /Lib
parent61f8f58f812dab2c201a9c64ebe69e3271a6663c (diff)
downloadcpython-fd8aafd64d23e7de51566c7099c704d48da86eea.zip
cpython-fd8aafd64d23e7de51566c7099c704d48da86eea.tar.gz
cpython-fd8aafd64d23e7de51566c7099c704d48da86eea.tar.bz2
[3.12] gh-77749: Fix inconsistent behavior of non-ASCII handling in EmailPolicy.fold() (GH-6986) (GH-114606)
It now always encodes non-ASCII characters in headers if utf8 is false. (cherry picked from commit 504334c7be5a56237df2598d338cd494a42fca4c) Co-authored-by: Rito Takeuchi <licht-t@outlook.jp> Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Diffstat (limited to 'Lib')
-rw-r--r--Lib/email/policy.py9
-rw-r--r--Lib/test/test_email/test_policy.py17
2 files changed, 25 insertions, 1 deletions
diff --git a/Lib/email/policy.py b/Lib/email/policy.py
index 611deb5..8816c84 100644
--- a/Lib/email/policy.py
+++ b/Lib/email/policy.py
@@ -210,8 +210,15 @@ class EmailPolicy(Policy):
self.refold_source == 'long' and
(lines and len(lines[0])+len(name)+2 > maxlen or
any(len(x) > maxlen for x in lines[1:])))
- if refold or refold_binary and _has_surrogates(value):
+
+ if not refold:
+ if not self.utf8:
+ refold = not value.isascii()
+ elif refold_binary:
+ refold = _has_surrogates(value)
+ if refold:
return self.header_factory(name, ''.join(lines)).fold(policy=self)
+
return name + ': ' + self.linesep.join(lines) + self.linesep
diff --git a/Lib/test/test_email/test_policy.py b/Lib/test/test_email/test_policy.py
index e87c275..c6b9c80 100644
--- a/Lib/test/test_email/test_policy.py
+++ b/Lib/test/test_email/test_policy.py
@@ -135,6 +135,23 @@ class PolicyAPITests(unittest.TestCase):
for attr, value in expected.items():
self.assertEqual(getattr(added, attr), value)
+ def test_fold_utf8(self):
+ expected_ascii = 'Subject: =?utf-8?q?=C3=A1?=\n'
+ expected_utf8 = 'Subject: á\n'
+
+ msg = email.message.EmailMessage()
+ s = 'á'
+ msg['Subject'] = s
+
+ p_ascii = email.policy.default.clone()
+ p_utf8 = email.policy.default.clone(utf8=True)
+
+ self.assertEqual(p_ascii.fold('Subject', msg['Subject']), expected_ascii)
+ self.assertEqual(p_utf8.fold('Subject', msg['Subject']), expected_utf8)
+
+ self.assertEqual(p_ascii.fold('Subject', s), expected_ascii)
+ self.assertEqual(p_utf8.fold('Subject', s), expected_utf8)
+
def test_fold_zero_max_line_length(self):
expected = 'Subject: =?utf-8?q?=C3=A1?=\n'