Skip to content

chore(deps): waive RUSTSEC-2026-0192 (ttf-parser unmaintained)#175

Merged
StefanSteiner merged 1 commit into
tableau:mainfrom
StefanSteiner:chore/waive-ttf-parser-advisory
Jun 30, 2026
Merged

chore(deps): waive RUSTSEC-2026-0192 (ttf-parser unmaintained)#175
StefanSteiner merged 1 commit into
tableau:mainfrom
StefanSteiner:chore/waive-ttf-parser-advisory

Conversation

@StefanSteiner

Copy link
Copy Markdown
Contributor

What

Waive RUSTSEC-2026-0192 (ttf-parser unmaintained) in both deny.toml and
.cargo/audit.toml, so the cargo-deny CI job goes green again.

Why

ttf-parser was flagged unmaintained on 2026-06-28. The advisory is
informational (the author has stepped away), not a vulnerability, and it has
no patched version, so it applies to every release of the crate.

ttf-parser is not a direct dependency and is never called from our code.
It comes in transitively only through plotters, which we use in hyperdb-mcp
for server-side PNG/SVG chart rendering:

ttf-parser 0.20.0 -> plotters 0.3.7 -> hyperdb-mcp

A literal "switch to skrifa" isn't possible: plotters has no skrifa backend,
and both of its font paths terminate at ttf-parser (the ttf default via
font-kit, and the ab_glyph feature via ab_glyph -> owned_ttf_parser -> ttf-parser). A version bump can't clear it either, since the advisory covers
all versions.

Given there's no runtime risk in our usage (rasterizing chart text from
operator-supplied query results), waiving it is the right call for now. This
mirrors the existing paste unmaintained waiver already in deny.toml.

Upstream tracking

Commented on the active plotters refactor that moves its font stack to
harfrust/skrifa/zeno: plotters-rs/plotters#736. The waiver comments in
both config files reference that issue. We'll drop the waiver once a plotters
release lands that retires the ttf-parser path.

Verification

  • cargo deny check advisories -> advisories ok
  • cargo audit -> no RUSTSEC-2026-0192
  • cargo test -p hyperdb-mcp --test chart_tests -> 37/37 pass (no regression)

ttf-parser is flagged unmaintained (informational, no patched version).
It is pulled in only transitively via plotters' chart-rendering font
backend; we never call it directly, so there is no runtime risk. Both of
plotters' font backends (ttf, ab_glyph) terminate at ttf-parser, so the
advisory cannot be cleared by a version bump.

Mirror the waiver in deny.toml and .cargo/audit.toml, matching the existing
'paste' unmaintained precedent. Tracking upstream at plotters-rs/plotters#736;
drop this waiver once plotters migrates its font stack (e.g. to skrifa).
@StefanSteiner StefanSteiner merged commit e048bc7 into tableau:main Jun 30, 2026
12 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