Skip to content

[Snyk] Security upgrade adm-zip from 0.5.18 to 0.6.1 - #24

Open
lane-formio wants to merge 1 commit into
mainfrom
snyk-fix-e195729f00502120007c3c51973b585b
Open

lane-formio wants to merge 1 commit into
mainfrom
snyk-fix-e195729f00502120007c3c51973b585b

Conversation

@lane-formio

Copy link
Copy Markdown
Contributor

snyk-top-banner

Snyk has created this PR to fix 1 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 Symlink Attack
SNYK-JS-ADMZIP-19276676
  696  

Breaking Change Risk

Merge Risk: Medium

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:

🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

@lane-formio

Copy link
Copy Markdown
Contributor Author

Merge Risk: Medium

This upgrade for adm-zip introduces a higher minimum Node.js version and contains behavioral changes that require verification.

Key Changes:

  • Minimum Node.js Version: The minimum required Node.js version has been raised to 14. The previous requirement was incorrectly listed as 12, though the code already implicitly needed a higher version.
  • Behavioral Change in extractEntryTo: When using extractEntryTo with maintainEntryPath set to false, the new version now preserves the subdirectory structure from the archive. Previously, it would flatten the paths, placing all files directly into the target directory. This change could break logic that relies on the old file-flattening behavior.
  • TypeScript Types: The package now includes built-in TypeScript definitions, so you can remove @types/adm-zip from your project.
  • Security Fixes: This version addresses a Denial of Service vulnerability (CVE-2026-39244) and hardens against object injection attacks.

Recommendation:
Verify that your project's runtime environment meets the new Node.js 14 requirement. Review any usage of extractEntryTo to ensure the new directory preservation behavior does not negatively impact your application's logic.

Source: GitHub Releases

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

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