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] Fix for 3 vulnerabilities #49

Merged
merged 1 commit into from
Sep 7, 2024

Conversation

OKEAMAH
Copy link
Owner

@OKEAMAH OKEAMAH commented Sep 7, 2024

snyk-top-banner

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

Snyk changed the following file(s):

  • server/tests-py/remote_schemas/nodejs/package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
high severity External Influence of Sphere Definition
SNYK-JS-APOLLOGATEWAY-7840416
medium severity Denial of Service (DoS)
SNYK-JS-GRAPHQL-5905181
medium severity Missing Release of Resource after Effective Lifetime
SNYK-JS-INFLIGHT-6095116

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:

🦉 Denial of Service (DoS)

Copy link

changeset-bot bot commented Sep 7, 2024

⚠️ No Changeset found

Latest commit: 66231a6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

We have skipped reviewing this pull request. Here's why:

  • It seems to have been created by a bot ('[Snyk]' found in title). We assume it knows what it's doing!
  • We don't review packaging changes - Let us know if you'd like us to change this.

Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/firebase-admin@5.13.1 environment, filesystem, network Transitive: eval, shell +164 54.2 MB firebase-ops

🚮 Removed packages: npm/@apollo/gateway@2.0.3), npm/@apollo/subgraph@2.0.4), npm/apollo-server@3.8.1), npm/graphql-tag@2.10.1), npm/graphql@16.5.0)

View full report↗︎

@OKEAMAH OKEAMAH merged commit b3050c1 into master Sep 7, 2024
10 checks passed
@OKEAMAH OKEAMAH deleted the snyk-fix-3070c29c742ca1861be59fd9a690e6dc branch September 7, 2024 04:01
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