Skip to content

fix: pin NLTK path security fixes - #247

Open
muscariello wants to merge 1 commit into
mainfrom
fix/security-nltk-pathsec
Open

muscariello wants to merge 1 commit into
mainfrom
fix/security-nltk-pathsec

Conversation

@muscariello

Copy link
Copy Markdown
Member

Summary

  • Pin the transitive NLTK dependency to upstream commit c06cd2645e3116156466bece11a9eb989c04afae, which contains the pathsec fixes for GHSA-8mgp-746c-j5xp / CVE-2026-81726.
  • Keep the lockfile reproducible until NLTK publishes a patched release.

Validation

  • uv lock --check passed.
  • Verified the installed NLTK source is the pinned commit and that TransitionParser and AveragedPerceptron model paths use pathsec guards.
  • pip-audit reports no findings other than the advisory being addressed by the unreleased upstream source fix.
  • NoA web-surfer import smoke test passed.

Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants