Skip to content

Comments

[Snyk] Fix for 4 vulnerabilities#89

Open
lgdlong wants to merge 1 commit intomainfrom
snyk-fix-b6ca4c0d3c2ebab6576bd9507cfccff8
Open

[Snyk] Fix for 4 vulnerabilities#89
lgdlong wants to merge 1 commit intomainfrom
snyk-fix-b6ca4c0d3c2ebab6576bd9507cfccff8

Conversation

@lgdlong
Copy link
Owner

@lgdlong lgdlong commented Jan 11, 2026

snyk-top-banner

Snyk has created this PR to fix 4 vulnerabilities in the pnpm dependencies of this project.

Snyk changed the following file(s):

  • apps/api/package.json
⚠️ Warning
Failed to update the pnpm-lock.yaml, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue Score
medium severity Allocation of Resources Without Limits or Throttling
SNYK-JS-AXIOS-12613773
  666  
high severity Uncaught Exception
SNYK-JS-MULTER-10773732
  649  
medium severity SQL Injection
SNYK-JS-TYPEORM-13746469
  576  
medium severity Prototype Pollution
SNYK-JS-JSYAML-13961110
  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:

🦉 Allocation of Resources Without Limits or Throttling
🦉 Prototype Pollution
🦉 Uncaught Exception
🦉 More lessons are available in Snyk Learn

Copilot AI review requested due to automatic review settings January 11, 2026 10:19
@vercel
Copy link

vercel bot commented Jan 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
dev-wiki Error Error Jan 11, 2026 10:19am

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 11, 2026

Important

Review skipped

Ignore keyword(s) in the title.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

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 is an automated security fix PR from Snyk that updates 4 dependencies in the API application to address high and medium severity vulnerabilities. However, the pnpm lockfile was not updated, which means the security fixes will not actually be applied.

Changes:

  • Upgraded axios from ^1.11.0 to ^1.12.0 to fix Allocation of Resources Without Limits vulnerability (score 666)
  • Upgraded @nestjs/platform-express from ^11.0.1 to ^11.1.5 to fix Uncaught Exception vulnerability in multer (score 649)
  • Upgraded typeorm from ^0.3.25 to ^0.3.26 to fix SQL Injection vulnerability (score 576)
  • Upgraded @nestjs/swagger from ^11.2.0 to ^11.2.2 to fix Prototype Pollution vulnerability in js-yaml (score 559)

Comment on lines +30 to +37
"@nestjs/platform-express": "^11.1.5",
"@nestjs/swagger": "^11.2.2",
"@nestjs/typeorm": "^11.0.0",
"@supabase/supabase-js": "^2.53.0",
"@types/bcrypt": "^5.0.2",
"@types/passport-google-oauth20": "^2.0.16",
"@types/pg": "^8.15.4",
"axios": "^1.11.0",
"axios": "^1.12.0",
Copy link

Copilot AI Jan 11, 2026

Choose a reason for hiding this comment

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

The package.json has been updated to fix security vulnerabilities, but the pnpm-lock.yaml file has not been updated accordingly. This means the actual installed dependencies will still be the vulnerable versions. The lockfile still contains:

  • axios@1.11.0 (instead of 1.12.0)
  • @nestjs/platform-express@11.1.3 (instead of 11.1.5)
  • @nestjs/swagger@11.2.0 (instead of 11.2.2)
  • typeorm@0.3.25 (instead of 0.3.26)

You must run pnpm install at the repository root to update the lockfile before merging this PR, otherwise the security vulnerabilities will remain unfixed.

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