Skip to content

[Snyk] Fix for 2 vulnerabilities - #103

Open
cdobru wants to merge 3 commits into
mainfrom
snyk-fix-185123910abb906c911a067082aca875
Open

[Snyk] Fix for 2 vulnerabilities#103
cdobru wants to merge 3 commits into
mainfrom
snyk-fix-185123910abb906c911a067082aca875

Conversation

@cdobru

@cdobru cdobru commented Jul 26, 2026

Copy link
Copy Markdown
Owner

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 Allocation of Resources Without Limits or Throttling
SNYK-JS-BRACEEXPANSION-18313044
  828  
high severity Inefficient Algorithmic Complexity
SNYK-JS-JSYAML-18313070
  828  

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:

🦉 Allocation of Resources Without Limits or Throttling

@cdobru

cdobru commented Jul 26, 2026

Copy link
Copy Markdown
Owner Author

Merge Risk: High

This upgrade includes a major and highly impactful migration for TypeORM from v0.2.25 to v1.0.0, which requires significant code refactoring and environment changes. The express-fileupload upgrade is considered low-risk.

typeorm@0.2.25 → typeorm@1.0.0 (HIGH RISK)

This is a substantial upgrade that spans two major breaking release cycles (0.3.0 and 1.0.0). Manual intervention is required.

Key Breaking Changes:

  • Environment: Support for Node.js versions below 20 has been dropped. The library now targets ES2023. [1, 8]
  • API Rewrite (DataSource): The entire connection management API has been rewritten. createConnection, getConnection, and other global functions have been removed. You must refactor your code to initialize and use a DataSource instance to get repositories and manage connections. [2, 4, 6]
  • Repository Methods: Methods like findOne(id) and findByIds() have been removed. They are replaced by a consistent object-based syntax, such as findOneBy({ id }). [4, 10]
  • Find Options: The syntax for find options like relations and select has changed from string arrays (e.g., relations: ["profile"]) to nested objects (e.g., relations: { profile: true }). [4]
  • Driver Changes: The mysql driver is no longer supported and has been replaced by mysql2. [1, 8]

Recommendation:
This upgrade cannot be completed without a dedicated refactoring effort. Use the official @typeorm/codemod tool to automate a significant portion of the required changes. [4, 5] Carefully review the official "Upgrading from 0.3 to 1.0" guide for manual steps and verification. [8]

express-fileupload@0.0.5 → express-fileupload@1.1.10 (LOW RISK)

While this is a major version jump, the most significant breaking change was reverted within this version range. The md5 property on the uploaded file object was changed to a function in v1.0.0 but was reverted to a checksum string in v1.1.1, matching the original behavior. [3] This version also includes important security fixes for vulnerabilities like Prototype Pollution (CVE-2020-7699). [17]

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

@cdobru

cdobru commented Jul 26, 2026

Copy link
Copy Markdown
Owner Author

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

cdobru commented Jul 26, 2026

Copy link
Copy Markdown
Owner Author

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.

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