Skip to content
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

gh-101819: Explicitly disallow pickle protocols 0 and 1 in _io #104370

Closed
wants to merge 4 commits into from

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented May 10, 2023

This is part of the preparations for isolating the _io module.

This is part of the preparations for isolating the _io module.
@erlend-aasland
Copy link
Contributor Author

erlend-aasland commented May 10, 2023

See #84258 (comment) and #85224 (comment), for more info

@vstinner
Copy link
Member

See #84258 (comment) and #85224 (comment), for more info

I'm not confortable to have to dig deep into the bug tracker to get reliable information about object serialization. Can it be written in the Python documentation instead?

Since these comments, I recall that @serhiy-storchaka made generic changes on base object type so it's no longer needed, no?

object.__reduce__() has now a default implementation which returns the string "NotImplemented". Is this change still needed?

@erlend-aasland
Copy link
Contributor Author

object.__reduce__() has now a default implementation which returns the string "NotImplemented". Is this change still needed?

Hm, perhaps we only need to change the _io tests.

@erlend-aasland
Copy link
Contributor Author

I believe you are right, @vstinner; there is no need for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants