chore: resolve open Dependabot security alerts (axios, brace-expansion, morgan)#9456
Merged
Merged
Conversation
Bumps three dependencies flagged by Dependabot, closing all 7 open alerts: - axios 1.16.0 -> 1.18.1 (catalog), closing 5 alerts: GHSA-gcfj-64vw-6mp9 (high, inherited proxy after interceptor config clone), GHSA-hcpx-6fm6-wx23, GHSA-mwf2-3pr3-8698, GHSA-f4gw-2p7v-4548, GHSA-xj6q-8x83-jv6g. Dependabot proposed 1.18.0; 1.18.1 is a pure bugfix release on top of it that also fixes runtime crashes and AxiosError circular-serialisation, so it is used instead. Supersedes PR #9447. - brace-expansion 5.0.6 -> 5.0.7 (override), closing GHSA-3jxr-9vmj-r5cp (high, DoS via exponential-time expansion of consecutive {} groups). - morgan -> 1.11.0 (new override), closing GHSA-4vj7-5mj6-jm8m (log forging via unneutralized control characters in :remote-user). Pulled in transitively by @react-router/serve. Verified: check:types 28/28, check:lint 16/16, build 16/16.
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated dependency version pins for ChangesDependency catalog updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
prateekshourya29
approved these changes
Jul 21, 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.
Description
Closes all 7 open Dependabot security alerts on the repo by bumping three pins in
pnpm-workspace.yaml. Only axios had an automated PR (#9447);brace-expansionandmorganhad none, so they were quietly sitting unpatched.axios1.16.0→1.18.1brace-expansion5.0.6→5.0.7morgan1.11.0Advisories resolved
axios— GHSA-gcfj-64vw-6mp9 (high, Node HTTP adapter can use an inherited proxy after interceptor config cloning), GHSA-hcpx-6fm6-wx23 (form serializermaxDepthbypass via{}metatoken), GHSA-mwf2-3pr3-8698 (HTTP/2 streamed uploads bypassmaxBodyLength), GHSA-f4gw-2p7v-4548 (NO_PROXYbypass for0.0.0.0local addresses), GHSA-xj6q-8x83-jv6g (prototype pollution — auth subfields can inject Basic auth)brace-expansion— GHSA-3jxr-9vmj-r5cp (high, DoS via exponential-time expansion of consecutive non-expanding{}groups)morgan— GHSA-4vj7-5mj6-jm8m (log forging via unneutralized control characters in:remote-user)Implementation notes
AxiosError#causecircular-JSON serialisation failures. It satisfies all five advisories, so there was no reason to land the strictly worse version.morganneeded a new override rather than a direct bump — it is not a direct dependency anywhere in the workspace. It arrives transitively through@react-router/serve@7.15.0, so an override is the only lever available without waiting upstream.The lockfile resolves to exactly one copy of each package — no vulnerable duplicates remain:
Type of Change
Screenshots and Media (if applicable)
N/A — dependency-only change, no user-facing UI surface.
Test Scenarios
Verified locally against the new lockfile:
pnpm check:types— 28/28 tasks passpnpm check:lint— 16/16 tasks passpnpm build— 16/16 tasks passWorth exercising in review, since axios 1.17/1.18 changed runtime behaviour:
FormDatamultipart) —maxDepthhandling and the metatoken fix changed serialisation of deeply nested payloads.validateStatus: undefined— 1.18.0 addstransitional.validateStatusUndefinedResolves;validateStatus: nullremains the explicit "accept every status" form.http:/https:URLs that omit//now reject withERR_INVALID_URLinstead of being coerced.apps/adminrequest logging via@react-router/serve— confirm morgan output is unchanged aside from control-character neutralisation in:remote-user.References
🤖 Generated with Claude Code
Summary by CodeRabbit