Skip to content

fix(cargo-deny): ignore unmaintained transitive advisories with rationale#3257

Closed
KooshaPari wants to merge 18 commits into
tailcallhq:mainfrom
KooshaPari:chore/cargo-deny-fork-transitive-ignores-20260504
Closed

fix(cargo-deny): ignore unmaintained transitive advisories with rationale#3257
KooshaPari wants to merge 18 commits into
tailcallhq:mainfrom
KooshaPari:chore/cargo-deny-fork-transitive-ignores-20260504

Conversation

@KooshaPari
Copy link
Copy Markdown

Adds 4 ignore entries for bincode 1.x, paste, rustls-pemfile, yaml-rust — all transitive via upstream forgecode workspace deps. Resolution depends on upstream tailcallhq/forgecode bumps. Fork-specific RUSTSEC-2026-* ignores preserved. cargo deny check advisories now PASSes.

KooshaPari and others added 18 commits May 1, 2026 20:06
- Bootstrap deny.toml with license allowlist + advisory ignores
- Add license = MIT to workspace.package (was missing)
- Add license.workspace = true to all 27 crate manifests
- Ignore transitive unmaintained (bincode, yaml-rust, paste, rustls-pemfile)
- Ignore transitive vulns (hickory-proto, rustls-webpki) via aws-sdk/reqwest

Co-authored-by: Claude Opus 4.7 <[email protected]>
Pin all action refs to immutable SHAs across workflow files:
- checkout@v4 → @11bd71901bbe5b1630ceea73d27597364c9af683
- checkout@v6 → @de0fac2e4500dabe0009e67214ff5f5447ce83dd
- setup-node@v4/v5, setup-python@v4/v5, setup-go@v5
- upload-artifact@v4/v7, download-artifact@v4
- cache@v3/v4, github-script@v7
- configure-pages@v5/v6, deploy-pages@v4/v5
- upload-pages-artifact@v3/v5, dependency-review-action@v4

Fixes version-tag normalization (add v4/v5 tags where missing).
Fixes double-SHA corruption artifacts from prior patching rounds.

Co-Authored-By: Claude Opus 4.7 <[email protected]>
Co-Authored-By: Claude Opus 4.7 <[email protected]>
Co-Authored-By: Claude Opus 4.7 <[email protected]>
Reduce README from 1124 to 169 lines (-85%).
Keep: project name, description, quickstart, usage examples, why forge,
installation, community, documentation link.
Add fork disclaimer pointing to upstream tailcallhq/forgecode.
Preserve all upstream content via pointer comment.

Co-Authored-By: Claude Opus 4.7 <[email protected]>
Reduce README from 1124 to 169 lines (-85%).
Keep: project name, description, quickstart, usage examples, why forge,
installation, community, documentation link.
Add fork disclaimer pointing to upstream tailcallhq/forgecode.
Preserve all upstream content via pointer comment.

Co-authored-by: Phenotype Agent <[email protected]>
Co-authored-by: Claude Opus 4.7 <[email protected]>
Co-authored-by: Phenotype Agent <[email protected]>
Verified resolved upstream; advisory no longer triggers.

Co-authored-by: Claude Opus 4.7 <[email protected]>
…nale

- Add 4 ignore entries for bincode 1.x (RUSTSEC-2025-0141), paste
  (2024-0436), rustls-pemfile (2025-0134), yaml-rust (2024-0320) — all
  transitive via upstream forgecode workspace deps; resolution depends on
  upstream tailcallhq/forgecode bumps.
- Pre-existing fork-specific RUSTSEC-2026-* ignores preserved.
- cargo deny check advisories: PASS.
Copilot AI review requested due to automatic review settings May 4, 2026 11:25
@github-actions github-actions Bot added the type: fix Iterations on existing features or infrastructure. label May 4, 2026
@KooshaPari KooshaPari closed this May 4, 2026
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ KooshaPari
❌ Phenotype Agent


Phenotype Agent seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@KooshaPari KooshaPari review requested due to automatic review settings May 4, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: fix Iterations on existing features or infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants