Skip to content
This repository has been archived by the owner on Jun 28, 2021. It is now read-only.

Error happened when using csv-stringify/lib/es5 #123

Open
alixiashu opened this issue Feb 19, 2021 · 3 comments
Open

Error happened when using csv-stringify/lib/es5 #123

alixiashu opened this issue Feb 19, 2021 · 3 comments
Labels

Comments

@alixiashu
Copy link

Describe the bug

When I try to use csv-stringify/lib/es5 to stringify an array, there is an error happened and csvStringify couldn't catch the error.

The error message:
_stream_readable.js:824 Uncaught (in promise) ReferenceError: process is not defined
at Stringifier.push.../../node_modules/.pnpm/readable-stream@3.6.0/node_modules/readable-stream/lib/_stream_readable.js.Readable.on (_stream_readable.js:824)
at stringify (index.js:737)

To Reproduce

I just use the code below:
csvStringify( [{a: '1'}], { header: true, columns: [{key: 'a'}] }, (e,output) => { console.log(e+output) } )

It will directly throw the error, csvStringify couldn't catch the error and put it in the csvStringify.Callback

Additional context

  1. IE11 didn't support csv-stringify/lib/browser, that's why I used csv-stringify/lib/es5;
  2. Package version: 5.6.1
@alixiashu alixiashu added the bug label Feb 19, 2021
@alixiashu
Copy link
Author

@wdavidw
Copy link
Member

wdavidw commented Feb 22, 2021

I think we need a combination of browser and es5. Let me have a look.

@wdavidw
Copy link
Member

wdavidw commented Feb 22, 2021

Could you try ./lib/browser again with version 5.6.2 that was just published, it is now compatible with ES5 (older browser).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants