Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Snyk] Security upgrade axios from 1.7.7 to 1.7.8 #3149

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dlobo
Copy link
Contributor

@dlobo dlobo commented Nov 28, 2024

snyk-top-banner

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

Snyk changed the following file(s):

  • package.json
  • yarn.lock

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.

Vulnerabilities that will be fixed with an upgrade:

Issue
medium severity Cross-site Scripting (XSS)
SNYK-JS-AXIOS-6671926

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 Scripting (XSS)

Copy link

@github-actions github-actions bot temporarily deployed to pull request November 28, 2024 03:13 Inactive
Copy link

cypress bot commented Nov 28, 2024

Glific    Run #5433

Run Properties:  status check passed Passed #5433  •  git commit 54bcc404a4 ℹ️: Merge 6909185ed1ea9d4b0b514232695f35c59a421a3b into 896ef1e55e73ec924d323865eb3c...
Project Glific
Branch Review snyk-fix-3f9d47949aec2d39b0cdcd8e08f56a90
Run status status check passed Passed #5433
Run duration 25m 48s
Commit git commit 54bcc404a4 ℹ️: Merge 6909185ed1ea9d4b0b514232695f35c59a421a3b into 896ef1e55e73ec924d323865eb3c...
Committer Donald A Lobo
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 169
View all changes introduced in this branch ↗︎

@kurund kurund added the on hold This is on hold label Dec 9, 2024
Copy link

codecov bot commented Dec 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.72%. Comparing base (896ef1e) to head (6909185).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3149   +/-   ##
=======================================
  Coverage   81.72%   81.72%           
=======================================
  Files         329      329           
  Lines       10563    10563           
  Branches     2235     2235           
=======================================
  Hits         8633     8633           
  Misses       1316     1316           
  Partials      614      614           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on hold This is on hold
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants