Skip to content

gh-92049: Forbid pickling constants re._constants.SUCCESS etc #92070

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.

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

Merged
merged 1 commit into from
Apr 30, 2022

Conversation

serhiy-storchaka
Copy link
Member

Previously, pickling did not fail, but the result could not be unpickled.

Previously, pickling did not fail, but the result could not be unpickled.
@serhiy-storchaka serhiy-storchaka added type-bug An unexpected behavior, bug, or error needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes labels Apr 30, 2022
@serhiy-storchaka serhiy-storchaka linked an issue Apr 30, 2022 that may be closed by this pull request
@serhiy-storchaka serhiy-storchaka merged commit 6d0d547 into python:main Apr 30, 2022
@miss-islington
Copy link
Contributor

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

@serhiy-storchaka serhiy-storchaka deleted the sre_constants-pickle branch April 30, 2022 10:03
@miss-islington
Copy link
Contributor

Sorry, @serhiy-storchaka, I could not cleanly backport this to 3.10 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 6d0d547033e295f91f05030322acfbb0e280fc1f 3.10

@miss-islington
Copy link
Contributor

Sorry @serhiy-storchaka, I had trouble checking out the 3.9 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 6d0d547033e295f91f05030322acfbb0e280fc1f 3.9

serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Apr 30, 2022
…S etc (pythonGH-92070)

Previously, pickling did not fail, but the result could not be unpickled..
(cherry picked from commit 6d0d547)

Co-authored-by: Serhiy Storchaka <[email protected]>
@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Apr 30, 2022
@bedevere-bot
Copy link

GH-92073 is a backport of this pull request to the 3.10 branch.

@serhiy-storchaka serhiy-storchaka removed their assignment Apr 30, 2022
serhiy-storchaka added a commit that referenced this pull request Apr 30, 2022
…H-92070) (GH-92073)

Previously, pickling did not fail, but the result could not be unpickled.
(cherry picked from commit 6d0d547)
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request May 1, 2022
…SUCCESS etc (pythonGH-92070) (pythonGH-92073)

Previously, pickling did not fail, but the result could not be unpickled.
(cherry picked from commit 6d0d547)
(cherry picked from commit e8ff3c9)

Co-authored-by: Serhiy Storchaka <[email protected]>
serhiy-storchaka added a commit that referenced this pull request May 1, 2022
…H-92070) (GH-92073) (GH-92102)

Previously, pickling did not fail, but the result could not be unpickled.
(cherry picked from commit 6d0d547)
(cherry picked from commit e8ff3c9)
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
… etc (pythonGH-92070) (pythonGH-92073) (pythonGH-92102)

Previously, pickling did not fail, but the result could not be unpickled.
(cherry picked from commit 6d0d547)
(cherry picked from commit e8ff3c9)
@hugovk hugovk removed the needs backport to 3.9 only security fixes label Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't pickle sre_constants
4 participants