Skip to content

[Snyk] Security upgrade axios from 1.7.5 to 1.18.0 - #160

Open
Dala0 wants to merge 1 commit into
mainfrom
snyk-fix-56a65b9b97770219e77dcda515c7bdf4
Open

[Snyk] Security upgrade axios from 1.7.5 to 1.18.0#160
Dala0 wants to merge 1 commit into
mainfrom
snyk-fix-56a65b9b97770219e77dcda515c7bdf4

Conversation

@Dala0

@Dala0 Dala0 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

snyk-top-banner

Snyk has created this PR to fix 8 vulnerabilities in the yarn dependencies of this project.

Snyk changed the following file(s):

  • frontend/package.json

Note for zero-installs users

If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

⚠️ Warning
Failed to update the yarn.lock, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue Score
medium severity Uncontrolled Recursion
SNYK-JS-AXIOS-18060730
  738  
medium severity Prototype Pollution
SNYK-JS-AXIOS-18065351
  738  
medium severity Allocation of Resources Without Limits or Throttling
SNYK-JS-AXIOS-18060165
  708  
medium severity Uncontrolled Recursion
SNYK-JS-AXIOS-18060167
  708  
medium severity Prototype Pollution
SNYK-JS-AXIOS-18060659
  708  
medium severity Prototype Pollution
SNYK-JS-AXIOS-18065349
  708  
medium severity Allocation of Resources Without Limits or Throttling
SNYK-JS-AXIOS-18065353
  708  
medium severity Prototype Pollution
SNYK-JS-AXIOS-18065355
  708  

Breaking Change Risk

Merge Risk: Medium

Notice: This assessment is enhanced by AI.


Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

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:

🦉 Allocation of Resources Without Limits or Throttling
🦉 Uncontrolled Recursion
🦉 Prototype Pollution

@Dala0

Dala0 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author

Merge Risk: Medium

This upgrade spans multiple minor versions and introduces several security-driven behavioral changes that require verification.

Key Changes:

  • Stricter URL Validation: Starting with v1.18.0, Axios will reject malformed http: and https: URLs that are missing the // separator, which may cause requests to fail if your application constructs URLs improperly. [1, 6]
  • Redirect Security: In v1.18.0, the Node.js adapter was updated to strip sensitive headers (like Authorization) on cross-origin redirects to prevent credentials from leaking. Applications that rely on forwarding these headers during a redirect may see a change in behavior. [1, 6]
  • Configuration Hardening: Multiple versions (including 1.17.0) added protections against prototype pollution by guarding how configuration options like socketPath, params, and paramsSerializer are read. This should not impact correctly configured applications but hardens security. [1, 2]

Recommendation: The risk is assessed as medium because these changes, while primarily for security, alter behavior. It is recommended to test application functionality related to URL construction, requests that involve redirects across different origins, and proxy configurations.

Source: GitHub Releases

Notice 🤖: This content was augmented using artificial intelligence. AI-generated content may contain errors and should be reviewed for accuracy before use.

@github-actions

Copy link
Copy Markdown

Unit Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit e518fc9.

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.

2 participants