FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-156026: document tar filter link targets by Amandot · Pull Request #156056 · python/cpython · GitHub

Repository navigation

gh-156026: document tar filter link targets - #156056

Merged
ethanfurman merged 1 commit into
python:mainfrom
Amandot:gh-156026-tar-filter-link-targets
Oct 12, 2026
Merged

ethanfurman merged 1 commit into
python:mainfrom
Amandot:gh-156026-tar-filter-link-targets

Conversation

Amandot commented Aug 19, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

Document the behavior of the tar extraction filter when handling
symbolic-link targets.

tar_filter protects against archive member paths escaping the extraction
destination, but it does not check TarInfo.linkname. As a result, symbolic
links with absolute targets or relative targets outside the destination are
not rejected by this filter.

This change clarifies that behavior in the tar filter documentation and
the tar_filter section.

The stricter data_filter behavior is also documented, including the
exceptions raised for unsafe link targets.

No extraction behavior has been changed.

Testing

  • git diff --check
  • Full documentation checks were attempted, but the documentation environment
    required dependency installation and the check was not completed.

Amandot requested a review from ethanfurman as a code owner August 19, 2026 14:46

python-cla-bot Bot commented Aug 19, 2026 •
edited
Loading

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #34139088 | 📁 Comparing bdb2a0e against main (20e6c2f)

  🔍 Preview build  

1 file changed
± library/tarfile.html

encukou commented Oct 8, 2026

Copy link
Copy Markdown
Member

I think this is much too wordy; see #159019.

ethanfurman merged commit e29cce5 into python:main Oct 12, 2026
40 checks passed
github-project-automation Bot moved this from Todo to Done in Docs PRs Oct 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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 skip news

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL