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 Issue with plotly.js #391

Closed
jiyuan12354 opened this issue Jan 10, 2024 · 3 comments
Closed

Security Issue with plotly.js #391

jiyuan12354 opened this issue Jan 10, 2024 · 3 comments

Comments

@jiyuan12354
Copy link

jiyuan12354 commented Jan 10, 2024

Firstly, I would like to express my gratitude to the SurveyJS team for providing such a robust open-source tool. Our company was so impressed that we didn’t hesitate to purchase the pro plan.

However, we’ve encountered a problem that we need your assistance with. Our company has a portal site that relies on survey-analytics@1.8.42. Our security team has discovered a security issue with this version.
Snipaste_2024-01-10_15-18-06

Snipaste_2024-01-10_15-10-42

Even after updating to the latest version of survey-analytics, which includes plotly.js@2.11.1, the issue persists as this version of plotly.js does not contain the necessary fix.

Is there any possibility of updating to plotly.js@2.25.2, which we believe has the required security fix? Alternatively, could you suggest any other methods to circumvent this security issue?

We look forward to your response and thank you in advance for your help.

I hope this helps! Let me know if you need further assistance.

refer to:
Fixed
Fix potential prototype pollution in plot API calls [#6703, 6704]

@tsv2013
Copy link
Member

tsv2013 commented Jan 30, 2024

The current version is 1.9.127. It depends on

"plotly.js-dist-min": "^2.11.1",

Probably you need to upgrade.

@jiyuan12354
Copy link
Author

The current version is 1.9.127. It depends on

"plotly.js-dist-min": "^2.11.1",

Probably you need to upgrade.

Got your point.. but as I described, Even after updating to the latest version of survey-analytics, which includes plotly.js@2.11.1, the issue persists as this version of plotly.js does not contain the necessary fix.

@tsv2013 tsv2013 closed this as completed in 53b3cc5 Feb 8, 2024
@tsv2013
Copy link
Member

tsv2013 commented Feb 8, 2024

I've updated dependency:

"plotly.js-dist-min": "^2.28.0",

@jiyuan12354 could you check it on your side and tell me whether it works for you?

@tsv2013 tsv2013 self-assigned this Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants