Skip to content

fix(deps): follow-redirects 1.16.2 (GHSA-mrwp-f3q8-3g9p) - #206

Merged
ndbroadbent merged 1 commit into
mainfrom
nathan/follow-redirects-1.16.2
Nov 1, 2026
Merged

ndbroadbent merged 1 commit into
mainfrom
nathan/follow-redirects-1.16.2

Conversation

@ndbroadbent

Copy link
Copy Markdown
Member

The nightly Security Scan's bun audit fails on web/bun.lock: follow-redirects 1.16.1 is affected by GHSA-mrwp-f3q8-3g9p (high), where custom credentials can follow a redirect. 1.16.2 fixes it.

follow-redirects comes in through axios. Axios's browser build swaps out its Node HTTP adapter, the only code that uses follow-redirects, so the web UI never included it. Nothing shipped changes and no release is needed. This clears the audit.

The change is the one lockfile entry, version and integrity hash. Re-resolving it with bun install would have moved about 100 other packages. bun install --frozen-lockfile accepts the edited lockfile, and scripts/check_js_audit.sh passes for web, docs and mock-oauth.

GHSA-mrwp-f3q8-3g9p (high): follow-redirects 1.16.0 and 1.16.1 could send custom credentials across a redirect. It
comes in through axios, whose browser build leaves out the Node HTTP adapter that uses it, so the SPA never ran it;
the nightly bun audit fails on the lockfile entry.
@deepsource-io

deepsource-io Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in dcfcc47...aa8458f on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Nov 1, 2026 3:30a.m. Review ↗
Go Nov 1, 2026 3:30a.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@ndbroadbent
ndbroadbent merged commit d8d0fc2 into main Nov 1, 2026
11 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