[Snyk] Security upgrade axios from 1.12.0 to 1.13.5#14
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedIgnore keyword(s) in the title. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| Ruby | Apr 5, 2026 1:17a.m. | Review ↗ | |
| Rust | Apr 5, 2026 1:17a.m. | Review ↗ | |
| JavaScript | Apr 5, 2026 1:17a.m. | Review ↗ | |
| Scala | Apr 5, 2026 1:17a.m. | Review ↗ | |
| Shell | Apr 5, 2026 1:17a.m. | Review ↗ | |
| Secrets | Apr 5, 2026 1:17a.m. | Review ↗ | |
| Terraform | Apr 5, 2026 1:17a.m. | Review ↗ | |
| Swift | Apr 5, 2026 1:17a.m. | Review ↗ | |
| SQL | Apr 5, 2026 1:17a.m. | Review ↗ | |
| Test coverage | Apr 5, 2026 1:17a.m. | Review ↗ | |
| C & C++ | Apr 5, 2026 1:17a.m. | Review ↗ | |
| C# | Apr 5, 2026 1:17a.m. | Review ↗ | |
| Ansible | Apr 5, 2026 1:17a.m. | Review ↗ |
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes. Give us feedback
There was a problem hiding this comment.
Pull Request Overview
This PR must not be merged in its current state. Despite the stated intent of upgrading axios for security, the implementation contains critical anomalies consistent with a supply chain attack. Specifically, the version of axios (1.13.5) and follow-redirects (1.15.11) are not present on the public npm registry, and the integrity hashes in the lockfile are mismatched. Additionally, there is an unrelated and potentially regressive downgrade of the React library. A clean manual upgrade of axios is required.
About this PR
- The lockfile regeneration includes unrelated additions (e.g., 'fsevents') and sub-dependency updates. While sub-dependency updates can be normal for Snyk PRs, in the context of the invalid primary versions identified, the entire lockfile should be treated as untrusted and regenerated from scratch.
Test suggestions
- Verify axios version is 1.13.5 in package.json
- Ensure 'react' version remains at ^19.2.3 or higher
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Ensure 'react' version remains at ^19.2.3 or higher
🗒️ Improve review quality by adding custom instructions
| "http-proxy-middleware": "^3.0.5", | ||
| "leaflet": "^1.9.4", | ||
| "react": "^19.2.3", | ||
| "react": "^19.1.0", |
There was a problem hiding this comment.
🔴 HIGH RISK
The 'react' dependency was downgraded from ^19.2.3 to ^19.1.0. This is out of scope for an axios security fix and may introduce regressions. This change should be reverted. Furthermore, pinning the version to an exact version would resolve the security warning regarding variant versions identified in quality scans.
| "@types/react": "^19.1.8", | ||
| "@types/react-dom": "^19.1.6", | ||
| "axios": "^1.12.0", | ||
| "axios": "^1.13.5", |
There was a problem hiding this comment.
🔴 HIGH RISK
CRITICAL SECURITY RISK: The versions for axios (1.13.5) and follow-redirects (1.15.11) do not exist on the public npm registry. The lockfile provides the integrity hash for version 1.15.9 while labeling it as 1.15.11. These anomalies strongly suggest a supply chain attack or major metadata corruption. Additionally, to resolve security warnings regarding variant versions, the dependency should be pinned to an exact, legitimate version (e.g., 1.7.8 or current stable).
Action: Revert these changes and run npm install axios@latest to upgrade to a legitimate version.
…duce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-AXIOS-15252993
c4ccd47 to
fcdce08
Compare
Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):
aq-dashboard/package.jsonaq-dashboard/package-lock.jsonVulnerabilities that will be fixed with an upgrade:
SNYK-JS-AXIOS-15252993
Important
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution