fix: sanitize options passed to HC methods

vaadin / vaadin-charts

Vaadin Charts is a feature-rich interactive graph library that answers the data visualization needs of modern web applications

https://vaadin.com/charts

Other

51 stars 24 forks source link

fix: sanitize options passed to HC methods #613

Closed DiegoCardoso closed 7 months ago

DiegoCardoso commented 7 months ago

Description

Apply the workaround described here on the methods that use options in Highcharts to sanitize them and prevent any unwanted script to be executed. We must do it like this because Highcharts cannot be updated to V9+ in Charts for V14.

Fixes vaadin/web-components#7294

Type of change

[X] Bugfix
[ ] Feature

DiegoCardoso commented 7 months ago

I need to fix the P3 build.

DiegoCardoso commented 7 months ago

Build is green again thanks to @yuriy-fix, so I am marking this PR ready for review again.