Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

replace csvtojson with convert-csv-to-json #12

Merged
merged 4 commits into from
May 5, 2024
Merged

replace csvtojson with convert-csv-to-json #12

merged 4 commits into from
May 5, 2024

Conversation

mgrybyk
Copy link
Collaborator

@mgrybyk mgrybyk commented May 5, 2024

  • tried papaparse. It's even worse, it contains a critical vulnerability 😮‍💨
  • convert-csv-to-json uses fs sync API but I guess it's okay for now

fixed #11

dist/index.js Fixed Show fixed Hide fixed
Copy link
Contributor

github-actions bot commented May 5, 2024

@mgrybyk mgrybyk changed the title try another csv to json libs replace csvtojson with convert-csv-to-json May 5, 2024
@mgrybyk mgrybyk merged commit 62befc3 into main May 5, 2024
3 checks passed
@mgrybyk mgrybyk deleted the csv-to-json branch May 5, 2024 21:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bluebird in node-csvtojson causes security issue
1 participant