build(deps): bump h2 to 4.4.1 and js-yaml to 4.3.1 - #36147
Merged
yuneng-berri merged 2 commits intoAug 7, 2026
Conversation
Closes GHSA-5p4m-2wfm-xmqj (CVSS 7.5), flagged by osv-scan against ui/litellm-dashboard/package-lock.json. js-yaml is pinned by an exact npm override, so the override and the lock move together. Dev-only dependency: js-yaml reaches the tree through eslintrc, knip and @redocly/openapi-core, none of which ship in the built dashboard. 4.3.1 published 2026-07-31, clear of the 3-day min-release-age cooldown.
Contributor
Greptile SummaryThis PR updates the dashboard’s pinned, development-only
Confidence Score: 5/5The PR appears safe to merge because the manifest and lockfile remain synchronized and no changed-code failure was identified The patch is limited to a matching development dependency override and lock entry update, while existing consumers remain reachable through the same dependency structure
|
| Filename | Overview |
|---|---|
| ui/litellm-dashboard/package.json | Updates the exact js-yaml override to 4.3.1, with no concrete compatibility or installation regression identified |
| ui/litellm-dashboard/package-lock.json | Synchronizes the locked js-yaml version, registry URL, and integrity metadata with the manifest override |
Reviews (1): Last reviewed commit: "build(deps-dev): bump js-yaml to 4.3.1" | Re-trigger Greptile
Closes GHSA-6hr6-w5qg-qmwg (CVSS 5.3), the second finding from the same osv-scan run as the js-yaml bump. Bundled here so the scan goes green in one merge instead of two PRs that each stay red on the other's finding. Re-derived with `uv lock --upgrade-package h2` rather than taking the Dependabot lock wholesale: that keeps the diff to the two packages that actually move (h2, plus hpack 4.2.0 which h2 4.4.1 requires) and leaves the `exclude-newer` snapshot a real timestamp. h2 4.4.1 published 2026-08-03, hpack 4.2.0 on 2026-06-23 — both clear of the 3-day exclude-newer window.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
5 tasks
5 tasks
yucheng-berri
approved these changes
Aug 7, 2026
yuneng-berri
enabled auto-merge
August 7, 2026 02:09
shin-berri
approved these changes
Aug 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clears both findings from the current osv-scan against
litellm_internal_staging, bundled into one PR so the scan goes green on a single merge.GHSA-6hr6-w5qg-qmwguv.lockGHSA-5p4m-2wfm-xmqjui/litellm-dashboard/package-lock.jsonh2
Re-derived with
uv lock --upgrade-package h2. Two packages move — h2, plushpack4.1.0 -> 4.2.0, which h2 4.4.1 requires. Diff is 14 lines.This deliberately does not reuse the lockfile from the Dependabot PR for the same advisory, which came in at 388 changed lines because it also rewrote
exclude-newerto the degenerate0001-01-01T00:00:00Zplaceholder and strippedpython_full_versionmarkers from ~190 dependency entries. Re-locking with the repo's own uv keepsexclude-newera real timestamp and leaves the markers alone.js-yaml
js-yamlis not a direct dependency — it arrives via@eslint/eslintrc,knipand@redocly/openapi-core, and is held at an exact version by an npmoverridesentry. The override and the lock entry therefore move together; a lock-only bump would be reverted by the next resolve.Verification
osv-scanner scan source --config osv-scanner.toml -L uv.lock -L ui/litellm-dashboard/package-lock.jsonon this branch, using v2.3.8 (the version the workflow pins): No issues found.uv lock --checkpasses.npm cisucceeds on the regenerated lock; installed tree resolvesjs-yaml@4.3.1, and the lockintegritymatches the registry value for 4.3.1 exactly.npm run lintis unchanged from base — the single pre-existing error (no-restricted-importson theantdimport invertexCredentialsUpload.ts) is present onlitellm_internal_stagingand untouched here.One finding stays suppressed rather than fixed:
GHSA-w8v5-vhqr-4h9v(diskcache unsafe pickle deserialization) is filtered byosv-scanner.tomluntil 2026-09-09. Upstream still has no fixed release — diskcache 5.6.3 from 2023 is the latest and the advisory covers everything through it. That ignore will need extending or replacing before it expires.