Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

@expo/plist latest version 0.0.18 is dependent on a potential security vulnerable version of xmldom #4569

Closed
chiragsoni81245 opened this issue Oct 14, 2022 · 1 comment · Fixed by #4571
Labels
🧹 chore help wanted Extra attention is needed

Comments

@chiragsoni81245
Copy link

Summary

@expo/plist version 0.0.18 package which is by default being used in expo-cli is dependent on a potential security-vulnerable version of xmldom which is "@xmldom/xmldom" "~0.7.0"

Environment

expo-env-info 1.0.5 environment info:
System:
OS: Windows 10 10.0.25217
Binaries:
Node: 16.13.2 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD
npm: 8.19.1 - C:\Program Files\nodejs\npm.CMD
IDEs:
Android Studio: AI-212.5712.43.2112.8609683
npmPackages:
expo: ~45.0.0 => 45.0.6
react: 17.0.2 => 17.0.2
react-dom: 17.0.2 => 17.0.2
react-native: 0.68.2 => 0.68.2
react-native-web: 0.17.7 => 0.17.7
Expo Workflow: managed

Please specify your device/emulator/simulator platform, model and version

Android 10

Error output

No response

Reproducible demo or steps to reproduce from a blank project

We got a notification from GitHub that one of the packages which is "@xmldom/xmldom" "~0.7.0" we are using is potentially vulnerable so as we checked its being used via @expo/plist here is a patched version of @xmldom/xmldom@^0.8.3 please update dependencies for @expo/plist

@chiragsoni81245 chiragsoni81245 added the needs review Issue is ready to be reviewed by a maintainer label Oct 14, 2022
@EvanBacon
Copy link
Contributor

EvanBacon commented Oct 27, 2022

Would you be willing to open a PR migrating to the new version of xmldom? Sorry I didn't see the existing contribution, will test and merge.

@EvanBacon EvanBacon reopened this Oct 27, 2022
@EvanBacon EvanBacon added 🧹 chore help wanted Extra attention is needed and removed needs review Issue is ready to be reviewed by a maintainer labels Oct 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🧹 chore help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants