]> git.ipfire.org Git - thirdparty/Python/cpython.git/commit
bpo-30681: Support invalid date format or value in email Date header (GH-22090)
authorGeorges Toth <georges@trypill.org>
Tue, 27 Oct 2020 00:31:06 +0000 (01:31 +0100)
committerGitHub <noreply@github.com>
Tue, 27 Oct 2020 00:31:06 +0000 (17:31 -0700)
commit303aac8c56609290e122eecc14c038e9b1e4174a
tree625fa73aa444a2c2ec59ec1bfebbc33108eafc82
parent8e3b9f92835654943bb59d9658bb52e1b0f40a22
bpo-30681: Support invalid date format or value in email Date header (GH-22090)

I am re-submitting an older PR which was abandoned but is still relevant, #10783 by @timb07.

The issue being solved () is still relevant. The original PR #10783 was closed as
the final request changes were not applied and since abandoned.

In this new PR I have re-used the original patch plus applied both comments from the review, by @maxking and @pganssle.

For reference, here is the original PR description:
In email.utils.parsedate_to_datetime(), a failure to parse the date, or invalid date components (such as hour outside 0..23) raises an exception. Document this behaviour, and add tests to test_email/test_utils.py to confirm this behaviour.

In email.headerregistry.DateHeader.parse(), check when parsedate_to_datetime() raises an exception and add a new defect InvalidDateDefect; preserve the invalid value as the string value of the header, but set the datetime attribute to None.

Add tests to test_email/test_headerregistry.py to confirm this behaviour; also added test to test_email/test_inversion.py to confirm emails with such defective date headers round trip successfully.

This pull request incorporates feedback gratefully received from @bitdancer, @brettcannon, @Mariatta and @warsaw, and replaces the earlier PR #2254.

Automerge-Triggered-By: GH:warsaw
Doc/library/email.errors.rst
Doc/library/email.utils.rst
Lib/email/_parseaddr.py
Lib/email/errors.py
Lib/email/headerregistry.py
Lib/email/utils.py
Lib/test/test_email/test_headerregistry.py
Lib/test/test_email/test_inversion.py
Lib/test/test_email/test_utils.py
Misc/NEWS.d/next/Library/2020-09-04-17-33-04.bpo-30681.LR4fnY.rst [new file with mode: 0644]