-
Notifications
You must be signed in to change notification settings - Fork 85
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
Some errors thrown as exceptions rather than being passed to parseXlsx's callback #14
Comments
I was trying this My app.js var express = require('express'); //Call Express module
app.listen(3003); But I'm getting error /home/local/user/Desktop/sample/node_modules/excel/node_modules/node-promise/promise.js:199
Error: invalid signature: 0x4acf92ad (at position: 0x29) How to fix this issue? My file is existing i crosschecked it. My xlsx file is in the same folder as app.js |
It's the issue with the old 'unzip' library version used - it doesn't support zip format used by LibreOffice and Google Docs for producing XLSX |
I'm running into this error as well. Would it be possible to get around this by using a different unzip library? |
I believe a new unzip lib is used already - could you confirm you use the most recent versions of excel module? If yes, could you post the error output here? |
+1 invalid signature: 0x20726176 /node_modules/excel/node_modules/node-promise/promise.js:204 /node_modules/excel/node_modules/unzip/lib/parse.js:63:13 |
I am able to build and install on the same development board using 3.1.2 I've sent a bug ticket to BCM. |
Unfortunately I'm not familiar enough with the promise library to be able to see how to fix this yet, but it seems that some errors get thrown as exceptions rather than being passed to the callback given to parseXlsx. For example, if you give parseXslx the path of a nonexistent file, an ENOENT exception is thrown. The intended behavior is (presumably?) for such errors to be passed on to the callback.
The text was updated successfully, but these errors were encountered: