Skip to content

bpo-30681: Support invalid date format or value #2254

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
timb07 wants to merge 6 commits into from
Closed

bpo-30681: Support invalid date format or value #2254

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
2bf3127
bpo-30681: Change error handling to return None in case of invalid date
timb07 Jun 17, 2017
e0b616a
bpo-30681: Handle invalid date in Date header
timb07 Jun 17, 2017
c59e0bf
bpo-30681: Add InvalidDateDefect
timb07 Jun 17, 2017
5c6cfaf
Revert "bpo-30681: Change error handling to return None in case of in…
timb07 Jun 17, 2017
8c8ac38
bpo-30681: Handle invalid date in Date header
timb07 Nov 28, 2018
96f8223
bpo-30681: Document failure modes of parsedate_to_datetime and add tests
timb07 Nov 28, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
bpo-30681: Handle invalid date in Date header
  • Loading branch information
@timb07
timb07 committed Nov 28, 2018
commit 8c8ac38a04fb810d5fbabf988feb590793adebbc
5 changes: 3 additions & 2 deletions Lib/email/headerregistry.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -301,8 +301,9 @@ def parse(cls, value, kwds):
return
if isinstance(value, str):
kwds['decoded'] = value
value = utils.parsedate_to_datetime(value)
if value is None:
try:
value = utils.parsedate_to_datetime(value)
except (ValueError, TypeError):
kwds['defects'].append(errors.InvalidDateDefect('Invalid date value or format'))
kwds['datetime'] = None
kwds['parse_tree'] = parser.TokenList()
Expand Down