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

Upgrade Highcharts - Cross Site Scripting vuln #462

Open
rjensen-r7 opened this issue Jun 9, 2020 · 3 comments
Open

Upgrade Highcharts - Cross Site Scripting vuln #462

rjensen-r7 opened this issue Jun 9, 2020 · 3 comments

Comments

@rjensen-r7
Copy link

Highcharts dependency needs to be upgraded to >= 8.1.1.

https://www.npmjs.com/advisories/1227
Overview
Versions of highcharts prior to 8.1.1 are vulnerable to Cross-Site Scripting (XSS). The package fails to sanitize href values and does not restrict URL schemes, allowing attackers to execute arbitrary JavaScript in a victim's browser if they click the link.

Remediation
Upgrade to version 8.1.1 or later.

@lsiler-mdsol
Copy link

lsiler-mdsol commented Jun 16, 2020

is anyone working on resolving this? @kirjs

plondon pushed a commit to plondon/react-highcharts that referenced this issue Jun 22, 2020
@piyalcodes
Copy link

I'm also getting the same problem even for the latest version

Version installed:
"highcharts": "^8.2.0",
"react-highcharts": "^16.1.0",

┌───────────────┬──────────────────────────────────────────────────────────────┐
│ High │ Cross-Site Scripting │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package │ highcharts │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in │ >=7.2.2 <8.0.0 || >=8.1.1 │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ react-highcharts │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path │ react-highcharts > highcharts │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info │ https://npmjs.com/advisories/1227
└───────────────┴──────────────────────────────────────────────────────────────┘

Hope this will get fix sooner. 👍

@murb
Copy link

murb commented Oct 21, 2020

There is an official HighchartsReact wrapper now, which might be the path forward: https://github.com/highcharts/highcharts-react

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

No branches or pull requests

4 participants