Skip to content

fix(deps): vuln axios (minor → 1.21.0) [example/package.json] - #1525

Closed
gh-worker-campaigns-3e9aa4[bot] wants to merge 1 commit into
developfrom
engraver-auto-version-upgrade/minorpatch/npm/example/1-1793607910
Closed

gh-worker-campaigns-3e9aa4[bot] wants to merge 1 commit into
developfrom
engraver-auto-version-upgrade/minorpatch/npm/example/1-1793607910

Conversation

@gh-worker-campaigns-3e9aa4

Copy link
Copy Markdown
Contributor

Summary: High-severity security update — 1 package upgraded (MINOR changes included)

Manifests changed:

  • example/package.json (yarn)

✅ Action Required: Please review the changes below. If they look good, approve and merge this PR.


Updates

Package From To Type Dep Type Vulnerabilities Fixed
axios 1.18.0 1.21.0 minor Direct 7 HIGH, 5 MEDIUM

Security Details

🚨 Critical & High Severity (7 fixed)
Package CVE Severity Summary Unsafe Version Fixed In Case
axios CVE-2026-101898 HIGH Axios: HTTP/2 adapter bypasses configured DNS lookup and proxy controls 1.18.0 1.20.0 -
axios CVE-2026-101901 HIGH Axios: Denial of Service via Unhandled 'error' Event in HTTP/2 ClientHttp2Session Initialization 1.18.0 1.20.0 -
axios CVE-2026-101903 HIGH Axios: ReDoS in fromDataURI data: URL parser freezes the Node event loop (DoS) 1.18.0 1.20.0 -
axios CVE-2026-101905 HIGH Axios: Node HTTP adapter prototype-pollution gadget allows request socket hijack via inherited createConnection 1.18.0 1.20.0 -
axios CVE-2026-101906 HIGH Axios: ReDoS (O(N²)) in shouldBypassProxy host normalization, reachable via untrusted redirect Location 1.18.0 1.20.0 -
axios CVE-2026-101907 HIGH Axios: maxRedirects: 0 is not enforced by the fetch adapter, allowing redirect-based SSRF 1.18.0 1.20.0 -
axios CVE-2026-101909 HIGH Axios: Prototype Pollution Gadget in axios toFormData Options 1.18.0 0.34.0 -
ℹ️ Other Vulnerabilities (5)
Package CVE Severity Summary Unsafe Version Fixed In Case
axios CVE-2026-101899 MEDIUM Axios: CIDR-form NO_PROXY entries are ignored, causing proxy exclusion bypass for internal IP ranges 1.18.0 1.20.0 -
axios CVE-2026-101900 MEDIUM Axios: Fetch Adapter Header Injection via Inherited FormData getHeaders 1.18.0 1.20.0 -
axios CVE-2026-101902 MEDIUM Axios: Prototype-Pollution Gadget in the Default Instance Allows Inherited Object.prototype.method to Override HTTP Method 1.18.0 0.34.0 -
axios CVE-2026-101904 MEDIUM Axios: Header Injection via Inherited headers After Minimal Interceptor 1.18.0 1.20.0 -
axios CVE-2026-101908 MEDIUM Axios: Prototype pollution gadget in fetch adapter can alter outbound requests 1.18.0 1.20.0 -

Review Checklist

Standard review:

  • Review changes for compatibility with your code
  • Check for breaking changes in release notes
  • Run tests locally or wait for CI
  • Approve and merge this PR

Update Mode: all_vulns

🤖 Generated by DataDog Automated Dependency Management System

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-02T20:55:48.994133Z 0dc5ab2 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@dd-prapprover-prod-77c48c

dd-prapprover-prod-77c48c Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

PRApprover will approve and merge this PR, FAQ, #dx-source-code-management

🛠️ PRApproval Status

  • ✅ PR is eligible for auto-approval by rule dependency-management-version-updater - 2026-11-03T08:07:59Z
  • ✅ CI tests passed - 2026-11-03T08:08:03Z
  • ✅ Approved (commit: 0dc5ab2) - 2026-11-03T08:08:06Z
  • ⬜ Merge Started
  • ⬜ Merged

➡️ Current phase: waiting to merge...

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The root Axios resolution still forces version 1.18.0 under Yarn 3, so the workspace pin does not install the security update; the root resolution and lockfile must be updated to 1.21.0.

1 open issue
What changed in this PR

This PR aims to update Axios in the example application to address the security vulnerabilities listed in the description.

Changes:

  • Raises the example’s Axios dependency from 1.20.0 to 1.21.0.
  • Updates the workspace dependency declaration in the lockfile and refreshes five unrelated dependency resolutions.
File Description
yarn.lock Updates the example’s Axios declaration and several unrelated locked versions.
example/​package.json Bumps the Axios dependency to 1.21.0.

Review effort: Balanced

Comment thread example/package.json
"@react-navigation/native": "^6.1.18",
"@react-navigation/stack": "^6.2.1",
"axios": "1.20.0",
"axios": "1.21.0",

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0dc5ab2973

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread example/package.json
"@react-navigation/native": "^6.1.18",
"@react-navigation/stack": "^6.2.1",
"axios": "1.20.0",
"axios": "1.21.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Update the global Axios resolution too

The root package.json still forces every Axios request to 1.18.0 through its resolutions entry, so this workspace dependency remains on the vulnerable release despite declaring 1.21.0; the resulting lockfile confirms this by containing only an axios@npm:1.18.0 package entry. Update the global resolution and regenerate the lockfile so installs actually receive the security fix.

Useful? React with 👍 / 👎.

@dd-prapprover-prod-77c48c dd-prapprover-prod-77c48c Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR has been automatically approved by the DD PR Approver bot.

@datadog-datadog-us1-prod datadog-datadog-us1-prod Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bits Code Review: FAIL

The root Yarn resolution still pins Axios to 1.18.0, so changing the example manifest to 1.21.0 does not install the intended security update.

Open Bits AI session

🤖 Bits Code Review · Commit 0dc5ab2

Comment thread example/package.json
"@react-navigation/native": "^6.1.18",
"@react-navigation/stack": "^6.2.1",
"axios": "1.20.0",
"axios": "1.21.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Update the root Axios resolution so the upgrade takes effect

Yarn installs still resolve Axios to 1.18.0 because the root package.json overrides every Axios dependency with resolutions.axios. The lockfile also contains only Axios 1.18.0, so this declaration does not apply the intended security update. Update the root resolution and regenerate the lockfile to install 1.21.0.

React: 👍 useful · 👎 wrong · 😕 unclear · 👀 right, but not acting on it

What each reaction means
  • 👍 Useful: correct, and you're fixing it or will.
  • 👎 Wrong: the finding is incorrect, for example the bug can't happen or the code was misread.
  • 😕 Unclear: you can't tell what it means or why it matters.
  • 👀 Right, not acting: correct, but intended, out of scope for this PR, a known risk, or too minor.

🤖 Bits Code Review · Open Bits AI session

@maciejburda
maciejburda requested a review from sbarrio November 3, 2026 08:07
@sbarrio sbarrio closed this Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants