Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NPM IP package vulnerable to Server-Side Request Forgery (SSRF) attacks #2673

Closed
ztefanie opened this issue Feb 15, 2024 · 1 comment · Fixed by #2693
Closed

NPM IP package vulnerable to Server-Side Request Forgery (SSRF) attacks #2673

ztefanie opened this issue Feb 15, 2024 · 1 comment · Fixed by #2693
Labels
not-testable Not testable by our tester Task

Comments

@ztefanie
Copy link
Member

Is your feature request related to a problem? Please describe.
All packages using the ip package should be updated, so the ip package version is > (greater not greater equal!) 2.0.0

Additional context
See here: https://github.com/digitalfabrik/integreat-app/security/dependabot/149
Packages that need updates are:

   - Hoisted from "_project_#native#react-native#@react-native-community#cli#@react-native-community#cli-doctor#ip"
   - Hoisted from "_project_#native#react-native#@react-native-community#cli#@react-native-community#cli-hermes#ip

Use yarn why ip to check if all occurences are updated correctly.

@ztefanie ztefanie added the Task label Feb 15, 2024
@LeandraH
Copy link
Contributor

From react-native-community/cli#2294 : FYI, the only affected command is profile-hermes when producing source maps. If you're not using it on a server (e.g. your CI), you're safe to ignore this and wait for us to patch it once we have a proper solution. If you are using it however, please disable it temporarily.

We don't use profile-hermes as far as I can tell, so we should be able to just wait for them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-testable Not testable by our tester Task
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants