Skip to content

[Snyk] Fix for 5 vulnerabilities#47

Open
greg-md wants to merge 1 commit intomainfrom
snyk-fix-4ec10c1e3dbe9b12dd3d64b376d94a0a
Open

[Snyk] Fix for 5 vulnerabilities#47
greg-md wants to merge 1 commit intomainfrom
snyk-fix-4ec10c1e3dbe9b12dd3d64b376d94a0a

Conversation

@greg-md
Copy link
Owner

@greg-md greg-md commented Feb 13, 2026

snyk-top-banner

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

Snyk changed the following file(s):

  • package.json
  • package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue Score
high severity Prototype Pollution
SNYK-JS-AXIOS-15252993
  828  
high severity Interpretation Conflict
SNYK-JS-FASTIFY-15182642
  828  
high severity Regular Expression Denial of Service (ReDoS)
SNYK-JS-AJV-15274295
  803  
medium severity Allocation of Resources Without Limits or Throttling
SNYK-JS-FASTIFY-15182641
  601  
medium severity Prototype Pollution
SNYK-JS-LODASH-15053838
  559  

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:

🦉 Regular Expression Denial of Service (ReDoS)
🦉 Prototype Pollution
🦉 Allocation of Resources Without Limits or Throttling

Copilot AI review requested due to automatic review settings February 13, 2026 20:26
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This Snyk PR attempts to remediate reported npm dependency vulnerabilities by upgrading several NestJS-related dependencies (notably GraphQL and Fastify integrations), which cascades into major-version shifts across the Nest/Fastify/GraphQL stack.

Changes:

  • Upgraded @nestjs/common to ^9.0.0.
  • Upgraded @nestjs/graphql to ^13.2.4.
  • Upgraded @nestjs/platform-fastify to ^11.1.13 and updated the lockfile accordingly.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.

File Description
package.json Bumps selected NestJS packages to newer majors to address Snyk-reported vulns.
package-lock.json Regenerated dependency tree reflecting the NestJS/Fastify/GraphQL upgrades.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +28 to 32
"@nestjs/graphql": "^13.2.4",
"@nestjs/microservices": "^7.6.12",
"@nestjs/platform-fastify": "^7.6.12",
"@nestjs/platform-fastify": "^11.1.13",
"@types/sinon": "^9.0.10",
"apollo-server-fastify": "^2.21.0",
Copy link

Copilot AI Feb 13, 2026

Choose a reason for hiding this comment

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

@nestjs/graphql@13.x no longer bundles an Apollo integration; it expects a GraphQL driver package (e.g., @nestjs/apollo) to be installed/used. This repo doesn’t currently depend on @nestjs/apollo, and still has apollo-server-fastify@2.x pinned, which is unlikely to be compatible with the upgraded Nest GraphQL stack. Add the appropriate Nest GraphQL driver dependency (and adjust GraphQLModule configuration accordingly), or pin @nestjs/graphql to a version compatible with the existing Apollo Server v2 setup.

Copilot uses AI. Check for mistakes.
Comment on lines +25 to +30
"@nestjs/common": "^9.0.0",
"@nestjs/config": "^0.6.3",
"@nestjs/core": "^7.5.1",
"@nestjs/graphql": "^7.9.11",
"@nestjs/graphql": "^13.2.4",
"@nestjs/microservices": "^7.6.12",
"@nestjs/platform-fastify": "^7.6.12",
"@nestjs/platform-fastify": "^11.1.13",
Copy link

Copilot AI Feb 13, 2026

Choose a reason for hiding this comment

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

The updated NestJS package versions are mutually incompatible: @nestjs/common@^9 is now out of sync with @nestjs/core@^7 / @nestjs/microservices@^7 / @nestjs/config@^0.6, and the upgraded @nestjs/platform-fastify@^11 + @nestjs/graphql@^13 require newer Nest majors. This will lead to npm peer-dependency resolution failures and/or runtime incompatibilities. Align all @nestjs/* packages (and their peers like rxjs) to a single compatible major version, or downgrade the upgraded packages to versions compatible with Nest v7.

Copilot uses AI. Check for mistakes.
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