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.

Sign up for GitHub

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

Jump to bottom

no licensing on one of exceljs dependencies #2686

Open
antoniobeltran opened this issue Feb 12, 2024 · 3 comments · May be fixed by #2687
Open

no licensing on one of exceljs dependencies #2686

antoniobeltran opened this issue Feb 12, 2024 · 3 comments · May be fixed by #2687

Comments

@antoniobeltran
Copy link

Not sure if this is the place to ask, but I'm using exceljs, and this uses buffers which is used by binary of which node-unzipper references. Security alarms went off stating that there is no license and the github accounts have been deleted. I'm wondering what your thoughts are on this.
@pnappa pnappa linked a pull request Feb 13, 2024 that will close this issue
Open
@pnappa
Copy link

pnappa commented Feb 13, 2024

I had a go at resolving this - I'm avoiding this package exactly for the same reason you mentioned here. It's too legally risky to use this library as it stands (even for open-source purposes).

@k0s15
Copy link

k0s15 commented Feb 29, 2024

Hi pnappa,

Any timeline for releasing this pull request?

Hi contributors,

Any contributors can help to review on the pull request?

@pnappa
Copy link

pnappa commented Feb 29, 2024

Hi @k0s15 , I'm not a maintainer of this repository. You can see my suggested pull request above (#2687). You'll have to wait for a maintainer to merge it.

@k0s15 k0s15 mentioned this issue Apr 15, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Breaking] Replace unzipper with yauzl-promise pnappa/exceljs
3 participants
@pnappa @k0s15 @antoniobeltran