Skip to content

deny: acknowledge unsafe-libyaml maintenance advisory (root) - #856

Merged
bfjelds (bfjelds) merged 1 commit into
mainfrom
user/bfjelds/deny-unsafe-libyaml-root
Oct 15, 2026
Merged

bfjelds (bfjelds) merged 1 commit into
mainfrom
user/bfjelds/deny-unsafe-libyaml-root

Conversation

@bfjelds

Copy link
Copy Markdown
Member

Suppress only RUSTSEC-2024-0449 in the root cargo-deny policy, mirroring tools/tailor/deny.toml (#853). serde_yaml still depends on the archived unsafe-libyaml; replacing the YAML parser needs separate work. Other advisories remain enforced.

Validated with cargo deny check advisories (RUSTSEC-2024-0449 no longer reported; remaining failures are pre-existing and unrelated to this change).

Suppress RUSTSEC-2024-0449 in the root cargo-deny policy, matching
tools/tailor/deny.toml (#853). serde_yaml still depends on the
archived unsafe-libyaml crate; replacing the YAML parser needs
separate work. Other advisories remain enforced.

Validated with cargo deny check advisories (RUSTSEC-2024-0449 no
longer reported).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: bf578387-c3f1-4356-b684-e768cfee0f93
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

@bfjelds
bfjelds (bfjelds) marked this pull request as ready for review October 14, 2026 22:14
@bfjelds
bfjelds (bfjelds) requested a review from a team as a code owner October 14, 2026 22:14
Copilot AI balanced review requested due to automatic review settings October 14, 2026 22:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The exception is narrowly scoped, uses supported syntax, and has no identified blocking issues.

0 open findings

What changed in this PR

Adds a narrowly scoped root cargo-deny exception for the unsafe-libyaml maintenance advisory, matching Tailor’s policy.

Changes:

  • Ignores only RUSTSEC-2024-0449, with a reason noting YAML parser migration.
  • Leaves other advisories enforced.
File Description
deny.toml Adds the advisory-specific exception and rationale.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bfjelds
bfjelds (bfjelds) merged commit 7016704 into main Oct 15, 2026
100 checks passed
@bfjelds
bfjelds (bfjelds) deleted the user/bfjelds/deny-unsafe-libyaml-root branch October 23, 2026 04:24
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