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

Use exceljs instead of xlsx #4317

Open
drernie opened this issue Feb 14, 2025 · 3 comments
Open

Use exceljs instead of xlsx #4317

drernie opened this issue Feb 14, 2025 · 3 comments
Assignees

Comments

@drernie
Copy link
Member

drernie commented Feb 14, 2025

https://stackoverflow.com/questions/76101101/xlsx-package-vulnerabilities-found-but-there-is-no-newer-package

As explained in the README, this project is no longer maintained on GitHub and no longer published to npm. Try to install from below link: https://cdn.sheetjs.com/xlsx-0.19.3/xlsx-0.19.3.tgz. In your package.json file add this:

"xlsx": "https://cdn.sheetjs.com/xlsx-0.19.3/xlsx-0.19.3.tgz"
@drernie
Copy link
Member Author

drernie commented Feb 14, 2025

@drernie drernie linked a pull request Feb 14, 2025 that will close this issue
@drernie drernie self-assigned this Feb 14, 2025
@drernie
Copy link
Member Author

drernie commented Feb 15, 2025

Considering all this, I think, we should invest time in moving to other library, for example, https://www.npmjs.com/package/exceljs.
We use it only for reading simple tabular data, and we use it through facade which have tests.
So, it shouldn't be difficult.

@drernie drernie changed the title Use newer version of xlsx Use exceljs instead of xlsx Feb 15, 2025
@drernie
Copy link
Member Author

drernie commented Feb 15, 2025

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.

1 participant