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

update xlsx to 0.19.3 to fix vulnerability #77

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

hakuta
Copy link
Contributor

@hakuta hakuta commented May 29, 2023

description

There is a vulnerability in SheetJS v0.18.5.
Additionally, this project is no longer maintained on GitHub and is not being published to npm anymore.
Therefore, I have changed the installation of SheetJS to be done via CDN.

ref

SheetJS/sheetjs#2822
https://docs.sheetjs.com/docs/getting-started/installation/nodejs

@MeHattabi
Copy link

@LuisEnMarroquin could you please approve?

@LuisEnMarroquin LuisEnMarroquin changed the base branch from main to develop September 27, 2023 15:35
@LuisEnMarroquin LuisEnMarroquin merged commit 45b45c3 into LuisEnMarroquin:develop Sep 27, 2023
1 check passed
@LuisEnMarroquin
Copy link
Owner

@MeHattabi approved and merged, ty

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 this pull request may close these issues.

None yet

3 participants