Skip to content

Hash-lock the auditor in the audit workflow - #6

Merged
elkaix merged 1 commit into
mainfrom
fix/hash-lock-auditor
Nov 2, 2026
Merged

elkaix merged 1 commit into
mainfrom
fix/hash-lock-auditor

Conversation

@elkaix

@elkaix elkaix commented Nov 2, 2026

Copy link
Copy Markdown
Contributor
  • audit installed pip-audit==2.10.1 with unpinned transitive deps; now installs from requirements-audit.txt (--require-hashes) and audits both locks, one run each (pip-audit rejects duplicates across files).
  • Probe: a lock with certifi 2026.7.22 makes the audit exit 1 (PYSEC-2026-4320); both real locks pass.
  • tasks/todo.md updated for PR Harden maintainer assurance from external review #4/Bump the actions group with 2 updates #5 merges. No package changes; required check names unchanged.

pip-audit was version-pinned but resolved its dependencies unpinned, so a
compromised transitive package could turn the audit falsely green. Lock it
in requirements-audit.txt and audit each lock in its own run, because
pip-audit rejects a package listed in two requirement files. Bring the
work ledger up to date with the merged PRs.
@coderabbitai

coderabbitai Bot commented Nov 2, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 17 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 32eed7fc-aceb-4ee0-a871-adc52222706c
Commits

Reviewing files that changed from the base of the PR and between aeb3b55 and 3300e5e.

Files selected for processing: 7
  • .github/workflows/audit.yml
  • AGENTS.md
  • CHANGELOG.md
  • README.md
  • requirements-audit.in
  • requirements-audit.txt
  • tasks/todo.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@elkaix
elkaix merged commit e9c9c0f into main Nov 2, 2026
8 checks passed
@elkaix
elkaix deleted the fix/hash-lock-auditor branch November 2, 2026 12:04
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