Skip to content

fix: bump xml2js to 0.6.2 - #129

Open
blimmer wants to merge 1 commit into
intacct:masterfrom
blimmer:bump-xml2js
Open

fix: bump xml2js to 0.6.2#129
blimmer wants to merge 1 commit into
intacct:masterfrom
blimmer:bump-xml2js

Conversation

@blimmer

@blimmer blimmer commented Apr 18, 2024

Copy link
Copy Markdown

Fixes #120

Resolves https://nvd.nist.gov/vuln/detail/CVE-2023-0842

0.6.2 is backward compatible with 0.4.x (0.5.x introduced a non-backward-compatible fix for the CVE listed above). Therefore, this should be safe to treat as a bugfix version (e.g., 2.2.2).

Comment thread package.json
"@types/node": "^14.14.41",
"@types/uuid": "^8.3.0",
"@types/xml2js": "^0.4.8",
"@types/xml2js": "^0.4.14",

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This is the latest version of the types package.

Comment thread package.json
"uuid": "^8.3.2",
"winston": "^3.3.3",
"xml2js": "^0.4.23",
"xml2js": "^0.6.2",

@blimmer blimmer Apr 18, 2024

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The vulnerability was technically fixed in 0.5.x, but it broke the .hasOwnProperty prototype method you're using in this codebase already.

Therefore, based on the repository owner's recommendation, I bumped to 0.6.x which fixes the CVE but in a backward compatible way with 0.4.x.

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.

Potential security issue: dependency on xml2js v0.4.x

1 participant