Skip to content

Conversation

@dignissimus
Copy link
Contributor

@dignissimus dignissimus commented Apr 22, 2022

Adds a warning to the documentation for shutil.unpack_archive noting that it is unsafe to unpack archives from untrusted sources. This is done in line with the documentation from Zipfile.extractall and TarFile.extractall.

Resolves #91783

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Apr 22, 2022
@miss-islington
Copy link
Contributor

Thanks @dignissimus for the PR, and @JelleZijlstra for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 2, 2022
…chive (pythonGH-91844)

(cherry picked from commit 4b297a9)

Co-authored-by: Sam Ezeh <sam.z.ezeh@gmail.com>
@bedevere-bot
Copy link

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

@bedevere-bot
Copy link

GH-92166 is a backport of this pull request to the 3.9 branch.

miss-islington added a commit that referenced this pull request May 2, 2022
…H-91844)

(cherry picked from commit 4b297a9)

Co-authored-by: Sam Ezeh <sam.z.ezeh@gmail.com>
miss-islington added a commit that referenced this pull request May 2, 2022
…H-91844)

(cherry picked from commit 4b297a9)

Co-authored-by: Sam Ezeh <sam.z.ezeh@gmail.com>
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
…chive (pythonGH-91844)

(cherry picked from commit 4b297a9)

Co-authored-by: Sam Ezeh <sam.z.ezeh@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Security] shutil unpack_archive docs should clarify the security implications

5 participants