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

Security warning: Dependabot alert: Got allows a redirect to a UNIX socket #446

Open
techterbium opened this issue Sep 22, 2022 · 2 comments
Assignees
Labels
dependencies Pull requests that update a dependency file

Comments

@techterbium
Copy link

Describe the bug
Security warning by dependabot alert: The got package before 11.8.5 and 12.1.0 for Node.js allows a redirect to a UNIX socket.

To Reproduce
happens on version 3.12.0

Expected behavior
A clear and concise description of what you expected to happen.

Logs
If applicable, add logs to help explain your problem.

** Operating system, Node.js an npm versions, or browser version (please complete the following information):**

  • OS: [e.g. Ubuntu 18.04]
  • Node.js: [e.g. 8.11.1]
  • npm: [e.g. 5.6.0]
  • Browser: [e.g. Chrome 73.0.3683]

Additional context
Add any other context about the problem here.

@javierbrea
Copy link
Member

Hi @techterbium ,
The "got" package is not a direct dependency of this project. So, you'll have to fix the security alert by pinning the dependency in your own package-lock.json file in your repository.

@javierbrea javierbrea added the stage: awaiting response Solution was proposed. Awaiting response. label Sep 22, 2022
@javierbrea javierbrea self-assigned this Sep 22, 2022
@FinnWoelm
Copy link

Hi @javierbrea,

first of all: Thanks for this great work!

Just wanted to jump in here and note that the security warning still exists on fresh install of mocks-server/main.

It appears that update-notifier (up to v5.1.0) depends on vulnerable version of the got package. And mocks-server/core depends on v5.1.0 of update-notifier.

image

"update-notifier": "5.1.0",

There is a v6 of update-notifier: https://github.com/yeoman/update-notifier/releases/tag/v6.0.0

We'd need to figure out if/how an upgrade to v6 affects mocks-server/core.

Otherwise, there is an alternative v5 version that has no breaking changes and fixes the vulnerability. See here: sindresorhus/update-notifier#218 (comment) Perhaps switching to update-notifier-cjs is an option?

@javierbrea javierbrea added dependencies Pull requests that update a dependency file and removed stage: awaiting response Solution was proposed. Awaiting response. labels May 3, 2023
@javierbrea javierbrea reopened this May 3, 2023
@javierbrea javierbrea moved this to In progress in Backlog May 20, 2024
@javierbrea javierbrea moved this from In progress to To do in Backlog May 20, 2024
@javierbrea javierbrea removed this from Backlog May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

No branches or pull requests

3 participants