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

End-Of-Stream error thrown parsing docx #313

Closed
Borewit opened this issue Jan 20, 2020 · 5 comments · Fixed by #316
Closed

End-Of-Stream error thrown parsing docx #313

Borewit opened this issue Jan 20, 2020 · 5 comments · Fixed by #316
Assignees
@Borewit
Labels
bug

Comments

@Borewit
Copy link
Collaborator

Borewit commented Jan 20, 2020

Reading the sample file from issue #103 UofTCSCoop.docx, an End-Of-Stream exception is thrown.

Related: #312
@Borewit Borewit added the bug label Jan 20, 2020
@Borewit Borewit self-assigned this Jan 20, 2020
@Borewit Borewit mentioned this issue Jan 20, 2020
Closed
@Borewit
Copy link
Collaborator Author

Borewit commented Jan 21, 2020

First get PR #315 in.

@alecgibson
Copy link

We're getting a similar thing on a .png image, although unfortunately I can't share the image.

@Borewit
Copy link
Collaborator Author

Borewit commented Jan 21, 2020

@alecgibson, I can prevent the error, I cannot fix the recognition of your PNG, for that I need a sample.
If you manage to find picture you can share, please open a new issue.

Borewit added a commit that referenced this issue Jan 21, 2020
@Borewit
Fix end-of-stream exception in PNG level 2 parser.
ecfa8f1 
Fixes #313.
@Borewit Borewit mentioned this issue Jan 21, 2020
Merged
Borewit added a commit that referenced this issue Jan 21, 2020
@Borewit
Fix end-of-stream exception in PNG level 2 parser.
e42bcf9 
Fixes #313.
sindresorhus pushed a commit that referenced this issue Jan 22, 2020
@Borewit @sindresorhus
Fix end-of-stream exception in PNG level 2 parser (#316)
79d9e25 
Fixes #313
@alecgibson
Copy link

@Borewit Looks like that's fixed our .png issue 👌 Thanks for the quick response! 🙌

@hafquangbaox
Copy link

hafquangbaox commented Feb 25, 2020 •
edited
Loading

I still get the error with the png file

node-| error Error: End-Of-Stream
node-|     at ReadStream.StreamReader.s.once (/home/node/xproject/node_modules/peek-readable/lib/index.js:28:46)
node-|     at Object.onceWrapper (events.js:286:20)
node-|     at ReadStream.emit (events.js:203:15)
node-|     at ReadStream.EventEmitter.emit (domain.js:448:20)
node-|     at endReadableNT (_stream_readable.js:1143:12)
node-|     at process._tickCallback (internal/process/next_tick.js:63:19)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Borewit Borewit

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix end-of-stream exception in PNG level 2 parser sindresorhus/file-type
3 participants
@hafquangbaox @alecgibson @Borewit