[data grid] Replace ExcelJS - Githubissues

mui / mui-x

MUI X: Build complex and data-rich applications using a growing list of advanced React components, like the Data Grid, Date and Time Pickers, Charts, and more!

https://mui.com/x/

3.9k stars 1.19k forks source link

[data grid] Replace ExcelJS #13267

Open joserodolfofreitas opened 1 month ago

joserodolfofreitas commented 1 month ago

Motivation

ExcelJS is not actively maintained and has many issues
- Security vulnerability coming from ExcelJS: https://github.com/mui/mui-x/issues/13187
- Fix 'unsafe-eval' CSP issue: https://github.com/exceljs/exceljs/pull/2672
- Replace unzipper: https://github.com/exceljs/exceljs/pull/2687
- Bundle size heavy but doesn't need to be: https://github.com/exceljs/exceljs/issues/2703

Alternatives

Fork ExcelJS and keep "external dependencies"
Replace external dependency with internal code

Search keywords:

michelengelen commented 1 month ago

Thanks @joserodolfofreitas 👍🏼

oliviertassinari commented 1 month ago

For what it's worth https://github.com/exceljs/exceljs/issues/2764

SCR-20240529-cjhs

See https://app.ashbyhq.com/home/upcoming/right-side/candidates/b3ff8b82-8c29-4aed-810b-1bede8f8d0e7/applications/0338b675-320f-480e-8b86-d591caabe9ef/feed