]> git.ipfire.org Git - thirdparty/Python/cpython.git/commit
gh-98188: Fix EmailMessage.get_payload to decode data when CTE value has extra text...
authorRanKKI <hliu86.me@gmail.com>
Mon, 6 Jan 2025 01:32:16 +0000 (12:32 +1100)
committerGitHub <noreply@github.com>
Mon, 6 Jan 2025 01:32:16 +0000 (20:32 -0500)
commita62ba52f1439c1f878a3ff9b8544caf9aeef9b90
tree1497bc9199ac370221eff8d042cdf4ff17e5d9a6
parent3b231be8f000ae59faa04d5a2f1af11beafee866
gh-98188: Fix EmailMessage.get_payload to decode data when CTE value has extra text (#127547)

Up to this point message handling has been very strict with regards to content encoding values: mixed case was accepted, but trailing blanks or other text would cause decoding failure, even if the first token was a valid encoding.  By Postel's Rule we should go ahead and decode as long as we can recognize that first token.  We have not thought of any security or backward compatibility concerns with this fix.

This fix does introduce a new technique/pattern to the Message code: we look to see if the header has a 'cte' attribute, and if so we use that.  This effectively promotes the header API exposed by HeaderRegistry to an API that any header parser "should" support.  This seems like a reasonable thing to do.  It is not, however, a requirement, as the string value of the header is still used if there is no cte attribute.

The full fix (ignore any trailing blanks or blank-separated trailing text) applies only to the non-compat32 API.  compat32 is only fixed to the extent that it now ignores trailing spaces.  Note that the HeaderRegistry parsing still records a HeaderDefect if there is extra text.

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Lib/email/message.py
Lib/test/test_email/test_email.py
Lib/test/test_email/test_headerregistry.py
Misc/ACKS
Misc/NEWS.d/next/Library/2024-12-03-14-45-16.gh-issue-98188.GX9i2b.rst [new file with mode: 0644]