Skip to content

[Snyk] Fix for 2 vulnerabilities#96

Open
cdobru wants to merge 3 commits intomainfrom
snyk-fix-5ec0f036605f723244de1f07a94b6093
Open

[Snyk] Fix for 2 vulnerabilities#96
cdobru wants to merge 3 commits intomainfrom
snyk-fix-5ec0f036605f723244de1f07a94b6093

Conversation

@cdobru
Copy link
Copy Markdown
Owner

@cdobru cdobru commented Mar 27, 2026

snyk-top-banner

Snyk has created this PR to fix 2 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 Infinite loop
SNYK-JS-BRACEEXPANSION-15789759
  555  
medium severity Regular Expression Denial of Service (ReDoS)
SNYK-JS-PATHTOREGEXP-15789761
  545  

Breaking Change Risk

Merge Risk: High

Notice: This assessment is enhanced by AI.


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)

@cdobru
Copy link
Copy Markdown
Owner Author

cdobru commented Mar 27, 2026

Merge Risk: High

This upgrade includes a major version increase for express-fileupload which introduces a breaking change, and a minor version bump for express.

express-fileupload@0.0.5 → 1.1.10

Risk: HIGH

This is a major version upgrade with a significant breaking change in version 1.0.0 related to how MD5 checksums are handled.

  • Breaking Change: The md5 property on the uploaded file object was changed from a string (the checksum) to a function (to compute the checksum) in version 1.0.0. Although this was reverted in version 1.1.1 to be a string value again, applications relying on the pre-1.0.0 behavior would have been broken.

Recommendation:
Verify if your application uses the req.files.yourfile.md5 property. If it does, ensure it is compatible with the behavior in version 1.1.10, where it is a string value representing the checksum.

express@4.12.4 → 4.22.0

Risk: LOW

This is a minor version upgrade within the Express 4.x series. The updates between these versions consist of bug fixes, security patches, and performance improvements. There are no documented breaking API changes for this version range.

Notice 🤖: This content was augmented using artificial intelligence. AI-generated content may contain errors and should be reviewed for accuracy before use.

@cdobru
Copy link
Copy Markdown
Owner Author

cdobru commented Mar 27, 2026

Snyk checks have failed. 13 issues have been found so far.

Status Scan Engine Critical High Medium Low Total (13)
Open Source Security 0 5 8 0 13 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@cdobru
Copy link
Copy Markdown
Owner Author

cdobru commented Mar 27, 2026

Snyk checks have failed. 13 issues have been found so far.

Status Scan Engine Critical High Medium Low Total (13)
Open Source Security 0 5 8 0 13 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@bojtetami
Copy link
Copy Markdown

bojtetami commented Mar 27, 2026

Snyk checks have failed. 13 issues have been found so far.

Status Scan Engine Critical High Medium Low Total (13)
Open Source Security 0 5 8 0 13 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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.

3 participants