Skip to content

ci: scan dependencies weekly - #9

Open
nothingmuch wants to merge 1 commit into
mainfrom
scheduled-flake-update
Open

ci: scan dependencies weekly#9
nothingmuch wants to merge 1 commit into
mainfrom
scheduled-flake-update

Conversation

@nothingmuch

@nothingmuch nothingmuch commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Run a read-only vulnerability scan weekly and on manual dispatch.

Add cargo-audit as a reproducible flake check backed by a pinned RustSec advisory database. The scheduled job recreates flake inputs for the two targeted tasks, runs that audit, builds the default package, and scans its runtime closure with vulnix.

The workflow requires no repository-writing token. Its action references are intentionally left unpinned for the subsequent actions-pinning PR to handle consistently.

@yuval-block
yuval-block force-pushed the scheduled-flake-update branch 5 times, most recently from 6ef65d9 to 86d350f Compare July 23, 2026 01:38
@yuval-block yuval-block changed the title ci: update flake.lock weekly ci: scan dependencies weekly Jul 23, 2026
@yuval-block
yuval-block force-pushed the scheduled-flake-update branch 2 times, most recently from dfdab6c to f7555f5 Compare July 23, 2026 01:45
@yuval-block
yuval-block marked this pull request as ready for review July 23, 2026 01:49
@yuval-block
yuval-block force-pushed the scheduled-flake-update branch 2 times, most recently from d448aab to c228628 Compare July 23, 2026 02:02
@yuval-block
yuval-block force-pushed the scheduled-flake-update branch from c228628 to abb31de Compare July 23, 2026 02:03
name: Vulnerability scan
on:
schedule:
- cron: "0 0 * * *"

@Mshehu5 Mshehu5 Jul 23, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CACK code looks good ran nix build '.#checks.x86_64-linux.cargo-audit' no error outputs was shown.

The PR description says this scan should run weekly but 0 0 * * * runs every day.
A comment will also help readabilty

Suggested change
- cron: "0 0 * * *"
schedule:
# Run weekly on Sunday at 00:00 UTC.
- cron: "0 0 * * 0"

@Mshehu5

Mshehu5 commented Jul 23, 2026

Copy link
Copy Markdown

Also tested cron job on my branch https://github.com/Mshehu5/concurrent-psbt/actions/runs/30013622141/job/89227795093

ran out off cache though but scan nix closures seems to have caught some packages

edit : seems some might also be false positives and not comfirmed

edit 2: after running locally this what I got
concurrent-psbt % nix develop -c vulnix --closure ./result
WARNING:ZODB.FileStorage:Ignoring index for /Users/shehu/.cache/vulnix/Data.fs
Found no advisories. Excellent!

- uses: DeterminateSystems/nix-installer-action@main
- uses: DeterminateSystems/magic-nix-cache-action@main
- name: Update vulnerability databases
run: nix flake update

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be nix flake update rustsec-advisory-db? a bare update refreshes everything, so vulnix scans the updated closure rather than the pinned one

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch

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.

3 participants