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

Fix/package vulnerability api #759

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

medtaher123
Copy link
Contributor

@medtaher123 medtaher123 commented Feb 14, 2025

Motivation

nstalling dependencies in the /api folder showed multiple security vulnerabilities. I ran npm audit fix to resolve most of them, but 3 warnings remain. I did not use --force to avoid potential breaking changes.

Fixes #758

Type of change:

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have performed a self-review of my own code

Notes

  • 3 warnings remain. We need to decide whether to use npm audit fix --force, which may introduce breaking changes.
  • These changes need testing by others to ensure there are no unexpected issues after the dependency updates. I’ve already tested them on my side.

@yassinedorbozgithub yassinedorbozgithub added the security Vulnerabilities, exploits, sensitive data label Feb 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security Vulnerabilities, exploits, sensitive data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🤔 [ISSUE] - Package Vulnerability api
2 participants