Skip to content

fix(security): resolve Dependabot high/critical alerts on v3.12.5#134

Merged
igoroctaviano merged 1 commit into
masterfrom
fix/security-dependabot-v3.12.5
Jul 6, 2026
Merged

fix(security): resolve Dependabot high/critical alerts on v3.12.5#134
igoroctaviano merged 1 commit into
masterfrom
fix/security-dependabot-v3.12.5

Conversation

@igoroctaviano

Copy link
Copy Markdown
Collaborator

Summary

  • No OHIF version bump neededmaster is already on OHIF v3.12.5 (merged via chore: update to lts stable ohif (3.12.5) #132 on 2026-06-26).
  • OHIF v3.12.5 has no dedicated security patch branch; upstream fixes for post-release CVEs land on master (3.13 beta). This PR backports targeted dependency patches while remaining on the v3.12.5 baseline.
  • Addresses remaining high/critical Dependabot alerts not fully covered by v3.12.5 resolutions:
    • Root yarn.lock: sigstore, fast-uri, minimatch, picomatch
    • platform/cli: align declared axios with resolution (1.17.0)
    • platform/docs: expand resolutions and refresh separate yarn.lock (lodash, node-forge, path-to-regexp, fast-uri, picomatch, shell-quote, ws, @babel/plugin-transform-modules-systemjs)

Context

Repo High/Critical open alerts
OHIF/Viewers @ v3.12.5 11 (axios — declared version in platform/cli lags resolution)
ImagingDataCommons/ViewersV3 @ master 32 (above + docs lockfile + newer transitive CVEs)

OHIF v3.12.5 release notes document the initial security dependency update (OHIF#6086). Newer advisories (e.g. sigstore, fast-uri, minimatch, picomatch) appeared after that release.

Test plan

  • yarn install --frozen-lockfile succeeds in repo root
  • yarn install --frozen-lockfile succeeds in platform/docs
  • Dependabot re-scan shows reduction in high/critical alerts after merge
  • Smoke test viewer build (yarn build or CI)

Patch transitive dependencies via yarn resolutions and lockfile updates
while staying on OHIF v3.12.5. Aligns platform/cli axios declaration
with the existing 1.17.0 resolution and updates platform/docs lockfile.
@igoroctaviano
igoroctaviano merged commit 88d165d into master Jul 6, 2026
1 check failed
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