project-zot / zui

UI for zot registry
Apache License 2.0
13 stars 21 forks source link

[Snyk] Security upgrade axios from 0.24.0 to 1.6.3 #441

Closed rchamarthy closed 2 months ago

rchamarthy commented 3 months ago

snyk-top-banner

Snyk has created this PR to fix 2 vulnerabilities in the npm dependencies of this project.

Snyk changed the following file(s):

Vulnerabilities that will be fixed with an upgrade:

Issue Score
high severity Cross-site Request Forgery (CSRF)
SNYK-JS-AXIOS-6032459
  676  
medium severity Regular Expression Denial of Service (ReDoS)
SNYK-JS-AXIOS-6124857
  586  

[!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:

πŸ¦‰ Cross-site Request Forgery (CSRF) πŸ¦‰ Regular Expression Denial of Service (ReDoS)

andaaron commented 3 months ago

Hi @raulkele, can you please check why the tests are failing for this upgrade?

raulkele commented 2 months ago

Hi @raulkele, can you please check why the tests are failing for this upgrade?

opened #442 with other dependency bumps included. It's still WIP as I'm having some trouble reproducing the failing integration test, on local env they all pass. So some investigation is still needed on my part.

andaaron commented 2 months ago

Hi @raulkele that you for looking into it. I think the integration test failure may not be related to the upgrade, I think I saw the failure starting to reproduce before that.

andaaron commented 2 months ago

Done in https://github.com/project-zot/zui/pull/442