]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
[3.15] gh-152060: Fix `_pydatetime.fromisoformat()` raising `AssertionError` on inval...
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Wed, 24 Jun 2026 14:10:54 +0000 (16:10 +0200)
committerGitHub <noreply@github.com>
Wed, 24 Jun 2026 14:10:54 +0000 (14:10 +0000)
(cherry picked from commit ff781d52d451db56e154aac35ae7f2c41b1695a4)

Co-authored-by: tonghuaroot (童话) <tonghuaroot@gmail.com>
Lib/_pydatetime.py
Lib/test/datetimetester.py
Misc/NEWS.d/next/Library/2026-06-24-10-46-35.gh-issue-152060.f2asrt.rst [new file with mode: 0644]

index c1448374402de4a99e154b5724f779f67cbcede9..db4ea8d30c7064ff36d4af6a36e936fdc3db0b78 100644 (file)
@@ -358,7 +358,8 @@ def _find_isoformat_datetime_separator(dtstr):
 def _parse_isoformat_date(dtstr):
     # It is assumed that this is an ASCII-only string of lengths 7, 8 or 10,
     # see the comment on Modules/_datetimemodule.c:_find_isoformat_datetime_separator
-    assert len(dtstr) in (7, 8, 10)
+    if len(dtstr) not in (7, 8, 10):
+        raise ValueError("Invalid isoformat string")
     year = int(dtstr[0:4])
     has_sep = dtstr[4] == '-'
 
index 1cbe78c1ecbfdc62e740a7231afe996c6807e1de..e29f5e3ecb5fd4f7a56c6ae200a6a4a78f69cff7 100644 (file)
@@ -3757,6 +3757,7 @@ class TestDateTime(TestDate):
             '2009-04-19T12:30:45+00:00:90', # Time zone field out from range
             '2009-04-19T12:30:45-00:90:00', # Time zone field out from range
             '2009-04-19T12:30:45-00:00:90', # Time zone field out from range
+            '2020-2020',                    # Ambiguous 9-char date portion
         ]
 
         for bad_str in bad_strs:
diff --git a/Misc/NEWS.d/next/Library/2026-06-24-10-46-35.gh-issue-152060.f2asrt.rst b/Misc/NEWS.d/next/Library/2026-06-24-10-46-35.gh-issue-152060.f2asrt.rst
new file mode 100644 (file)
index 0000000..6088a6b
--- /dev/null
@@ -0,0 +1,3 @@
+Fix :meth:`datetime.datetime.fromisoformat` raising :exc:`AssertionError`
+instead of :exc:`ValueError` for some malformed strings in the pure-Python
+implementation, matching the C implementation.