➜
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2019-06-22 16:15 byJustin Blanchard, last changed2022-04-11 14:59 byadmin. This issue is nowclosed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 14307 | merged | python-dev,2019-06-22 16:42 | |
| PR 15583 | merged | miss-islington,2019-08-29 07:36 | |
| PR 15584 | merged | miss-islington,2019-08-29 07:37 | |
| Messages (5) | |||
|---|---|---|---|
| msg346284 -(view) | Author: Justin Blanchard (Justin Blanchard)* | Date: 2019-06-22 16:15 | |
Underbpo-22005, Python 3 gained support for unpickling datetime data from Python 2. (Thanks!) It turns out the input validation isn't quite right: it bombs on datetime.time when the encoded seconds (not hours) field is >=24:python2>>> datetime.time(1, 2, 3).__reduce__()(<type 'datetime.time'>, ('\x01\x02\x03\x00\x00\x00',))python2>>> datetime.time(23, 24, 25).__reduce__()(<type 'datetime.time'>, ('\x17\x18\x19\x00\x00\x00',))python3>>> datetime.time('\x01\x02\x03\x00\x00\x00')datetime.time(1, 2, 3)python3>>> datetime.time('\x17\x18\x19\x00\x00\x00')Traceback (most recent call last): File "<stdin>", line 1, in <module>TypeError: an integer is required (got type str)I would like to contribute a fix and will be opening a PR. This is my first contribution - any help and patience appreciated! | |||
| msg350743 -(view) | Author: Serhiy Storchaka (serhiy.storchaka)* | Date: 2019-08-29 07:36 | |
New changeset122376df550b71dd3bec0513c7483cc1714212fa by Serhiy Storchaka (Justin Blanchard) in branch 'master':bpo-37372: Fix error unpickling datetime.time objects from Python 2 with seconds>=24. (GH-14307)https://github.com/python/cpython/commit/122376df550b71dd3bec0513c7483cc1714212fa | |||
| msg350746 -(view) | Author: miss-islington (miss-islington) | Date: 2019-08-29 07:56 | |
New changesetd1d42bf4a404f092fe90fe8984481c507a64ef0a by Miss Islington (bot) in branch '3.8':bpo-37372: Fix error unpickling datetime.time objects from Python 2 with seconds>=24. (GH-14307)https://github.com/python/cpython/commit/d1d42bf4a404f092fe90fe8984481c507a64ef0a | |||
| msg350747 -(view) | Author: miss-islington (miss-islington) | Date: 2019-08-29 07:57 | |
New changeset6b50c10f675a9e8438024c5fcc592b0d38d8c62d by Miss Islington (bot) in branch '3.7':bpo-37372: Fix error unpickling datetime.time objects from Python 2 with seconds>=24. (GH-14307)https://github.com/python/cpython/commit/6b50c10f675a9e8438024c5fcc592b0d38d8c62d | |||
| msg350751 -(view) | Author: Serhiy Storchaka (serhiy.storchaka)* | Date: 2019-08-29 08:12 | |
Thank you for your contribution Justin! | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:17 | admin | set | github: 81553 |
| 2019-08-29 08:12:38 | serhiy.storchaka | set | status: open -> closed resolution: fixed messages: +msg350751 stage: patch review -> resolved |
| 2019-08-29 07:57:44 | miss-islington | set | messages: +msg350747 |
| 2019-08-29 07:56:09 | miss-islington | set | nosy: +miss-islington messages: +msg350746 |
| 2019-08-29 07:37:03 | miss-islington | set | pull_requests: +pull_request15260 |
| 2019-08-29 07:36:54 | miss-islington | set | pull_requests: +pull_request15259 |
| 2019-08-29 07:36:20 | serhiy.storchaka | set | messages: +msg350743 |
| 2019-06-22 17:53:28 | xtreak | set | nosy: +serhiy.storchaka |
| 2019-06-22 16:51:31 | xtreak | set | nosy: +belopolsky,p-ganssle versions: - Python 3.6 |
| 2019-06-22 16:42:42 | python-dev | set | keywords: +patch stage: patch review pull_requests: +pull_request14131 |
| 2019-06-22 16:15:13 | Justin Blanchard | create | |