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

High and medium vulnerabilities found in deps Engine.io + glob-parent. #1847

Closed
2 of 10 tasks
mejiaj opened this issue Feb 9, 2021 · 2 comments · Fixed by #1936
Closed
2 of 10 tasks

High and medium vulnerabilities found in deps Engine.io + glob-parent. #1847

mejiaj opened this issue Feb 9, 2021 · 2 comments · Fixed by #1936

Comments

@mejiaj
Copy link

mejiaj commented Feb 9, 2021

Issue details

Snyk scan found the following vulnerabilities with dependencies.

✗ High severity vuln found in engine.io@3.5.0, introduced via browser-sync@2.26.14
    Description: Denial of Service (DoS)
    Info: https://snyk.io/vuln/SNYK-JS-ENGINEIO-1056749
    From: browser-sync@2.26.14 > socket.io@2.4.0 > engine.io@3.5.0

✗ Medium severity vuln found in glob-parent@5.1.1, introduced via browser-sync@2.26.14
    Description: Regular Expression Denial of Service (ReDoS)
    Info: https://snyk.io/vuln/SNYK-JS-GLOBPARENT-1016905
    From: browser-sync@2.26.14 > chokidar@3.5.1 > glob-parent@5.1.1

Steps to reproduce/test case

Please provide necessary steps for reproduction of this issue, or better the
reduced test case (without any external dependencies)
.

Please specify which version of Browsersync, node and npm you're running

  • Browsersync [ 2.26.14 ]
  • Node [ 14.15.4 ]
  • Npm [ 6.14.9 ]

Affected platforms

  • linux
  • windows
  • OS X
  • freebsd
  • solaris
  • other (please specify which)

Browsersync use-case

  • API
  • Gulp
  • Grunt
  • CLI

If CLI, please paste the entire command below

N/A

for all other use-cases, (gulp, grunt etc), please show us exactly how you're using Browsersync

N/A

@cronon
Copy link

cronon commented Feb 10, 2021

I see engine.io already patched their library, unfortunately the patch leads to breaking changes so they publish it in version 4
socketio/engine.io#612

@lachieh
Copy link
Contributor

lachieh commented Feb 24, 2022

This is the original issue, but #1850 has more details and more people are following it

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 a pull request may close this issue.

3 participants