Skip to content
This repository has been archived by the owner on Dec 16, 2023. It is now read-only.

Images corrupted in Node.js 0.12 #27

Open
loris opened this issue Feb 13, 2015 · 4 comments
Open

Images corrupted in Node.js 0.12 #27

loris opened this issue Feb 13, 2015 · 4 comments

Comments

@loris
Copy link

loris commented Feb 13, 2015

When using node-passbook to generate and sign a pkpass file, the latest image being added is always corrupted (see below). This issue appeared when upgrading to Node.js 0.12. Downgrading to 0.10.x solve the problem. Any idea?

@AndrewBarba
Copy link
Contributor

I am seeing similar results on io.js 1.2.0 and Node 0.12

@AndrewBarba
Copy link
Contributor

This should be fixed in version 2.1

@loris
Copy link
Author

loris commented Mar 5, 2015

Great news, will test this ASAP, could you explain what the bug/fix was about? Only change I found was the removal of an event listener : 6fcf949#diff-9719ab3f52c425bd2475836dc44bc0c8L396

@AndrewBarba
Copy link
Contributor

To be perfectly honestly with you, I don't know if my explanation is the real reason it is fixed but I tried removing it because the whole point of a writable-stream is for the write method to be called with a chunk of data. That event listener was either double calling the write method or it was messing up the stream with that source.resume()

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

No branches or pull requests

2 participants