Skip to content

Fix Security workflow audit scope - #49

Merged
rozling merged 1 commit into
mainfrom
fix/security-audit-project-dependencies
Sep 22, 2026
Merged

rozling merged 1 commit into
mainfrom
fix/security-audit-project-dependencies

Conversation

@rozling

@rozling rozling commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Why

The Security workflow audits its entire Python environment. Installing the Safety scanner also installs NLTK, whose unpatched PYSEC-2026-3740 advisory makes the job fail even though NLTK is not an application dependency.

Change

  • Audit requirements.txt with pip-audit in both the required check and failure report.
  • Remove the install step that was only needed for environment-wide auditing.
  • Run Security on pull requests targeting main so workflow changes are checked before merge.

Validation

  • actionlint .github/workflows/security.yml
  • .venv/bin/python -m pytest -v --tb=short (73 passed)
  • pre-commit run --all-files (all hooks passed)
  • pip-audit -r requirements.txt --format json --desc (PyYAML 6.0.3, no known vulnerabilities)

@rozling
rozling merged commit adf6d8e into main Sep 22, 2026
5 checks passed
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.

1 participant