Skip to content

fix: resolve Dependabot + code scanning alerts#10

Merged
aagarwal1012 merged 2 commits into
mainfrom
fix/security-alerts
Apr 23, 2026
Merged

fix: resolve Dependabot + code scanning alerts#10
aagarwal1012 merged 2 commits into
mainfrom
fix/security-alerts

Conversation

@aagarwal1012

Copy link
Copy Markdown
Member

Summary

Resolves all addressable open security alerts: 5 of 7 fixed (3 Dependabot, 3 code scanning), 2 upstream-blocked (documented below).

Dependabot (3 fixed, 2 upstream-blocked)

Alert Package Before After Severity GHSA
#2 time 0.3.45 0.3.47 medium GHSA-r6v5-fh4h-64xc
#3 rand (0.9.x) 0.9.2 0.9.3 low GHSA-cq8v-f236-94qc
#4 rand (0.8.x) 0.8.5 0.8.6 low GHSA-cq8v-f236-94qc

Upstream-blocked (will be dismissed with reason)

Alert Package Why it can't be fixed
#1 glib 0.18.5 Hard-pinned by gtk 0.18 which is hard-pinned by tauri 2.10.3. gtk-rs 0.18 branch is EOL/unmaintained. Tauri maintainers have marked this wontfix (tauri#12048). Fix requires Tauri's in-progress GTK4 migration (tauri#14684). Linux-only; macOS/Windows builds unaffected.
rand 0.7.3 Build-dep only (pulled by tauri-buildkuchikikiselectorsphf_codegen 0.8). No fix exists in the 0.7.x branch — it's abandoned. Does not ship in the compiled binary.

Code scanning (3 fixed)

Resolved all 3 actions/unpinned-tag warnings in .github/workflows/build.yml by pinning actions to full commit SHAs:

Alert Action Before After
#1 dtolnay/rust-toolchain @stable @29eef336d9b2848a0b548edc03f92a220660cdb8 (stable, 2026-03-27)
#2 swatinem/rust-cache @v2 @c19371144df3bb44fab255c43d04cbc2ab54d1c4 (v2.9.1)
#3 tauri-apps/tauri-action @v0 @84b9d35b5fc46c1e45415bdb6144030364f7ebc5 (action-v0.6.2)

Version comments follow GitHub's documented security pattern — Dependabot parses them to propose future version bumps.

Verification

  • cargo check --all-targets passes clean on macOS
  • LSP diagnostics clean on build.yml
  • No transitive version regressions introduced (only the alerted packages and their required compatibility updates: mac-notification-sys 0.6.10 → 0.6.12, notify-rust 4.12.0 → 4.16.0)
  • Full cross-platform CI build — will run automatically on this PR

How to test locally

cd src-tauri
cargo check --all-targets
cargo tree -i time:0.3.47    # confirm no 0.3.45 remains
cargo tree -i rand:0.8.6     # confirm no 0.8.5 remains
cargo tree -i rand:0.9.3     # confirm no 0.9.2 remains

Notes for reviewer

  • The 3 workflow action pins carry trailing # version comments — these are required per GitHub security docs and enable Dependabot's automated version bumps. Please do not strip them.
  • Two alerts are being dismissed after this PR merges (glib, rand 0.7.3) — both are upstream-blocked with no action available to this repo.

Resolves Dependabot alerts:
- time 0.3.45 -> 0.3.47  (GHSA-r6v5-fh4h-64xc, stack exhaustion DoS, medium)
- rand 0.8.5 -> 0.8.6    (GHSA-cq8v-f236-94qc, unsoundness, low)
- rand 0.9.2 -> 0.9.3    (GHSA-cq8v-f236-94qc, unsoundness, low)

Indirectly bumped mac-notification-sys 0.6.10 -> 0.6.12 and
notify-rust 4.12.0 -> 4.16.0 to unblock time's compatibility range.

cargo check passes clean on macOS.
Resolves CodeQL 'actions/unpinned-tag' warnings (medium security severity).
Pinning to immutable commit SHAs prevents supply-chain attacks via
compromised tag/branch references.

Pinned actions:
- dtolnay/rust-toolchain@stable      -> 29eef33 (stable @ 2026-03-27)
- swatinem/rust-cache@v2             -> c193711 (v2.9.1)
- tauri-apps/tauri-action@v0         -> 84b9d35 (action-v0.6.2)

Version comments follow GitHub's documented pattern; Dependabot
reads these to propose future version bumps.
@aagarwal1012 aagarwal1012 merged commit 004ea57 into main Apr 23, 2026
3 checks passed
@aagarwal1012 aagarwal1012 deleted the fix/security-alerts branch April 23, 2026 09:33
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