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.

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

better write stream #13

Merged
merged 1 commit into from
Jun 19, 2017
Merged

better write stream #13

merged 1 commit into from
Jun 19, 2017

Conversation

calvinmetcalf
Copy link
Contributor

great module, but the write stream doesn't quite follow the conventions of a regular write stream, this fixes it to make sure it acts like a normal write stream, I was having some issues where it would look like it was done but the data wasn't written yet

@jb55
Copy link
Owner

jb55 commented Jun 15, 2017

we removed this awhile back #5,#7 since we'd rather use the official stream upload implementation from aws-sdk. Perhaps there is a way to fix the issue you're referring to without the additional dependency?

@calvinmetcalf
Copy link
Contributor Author

the issue is just that since you have that proxy stream in the way, that one ends before it's actually done uploading and looking back through the aws skd it doesn't look like they expose a real stream

@calvinmetcalf
Copy link
Contributor Author

would it be helpful if I made a test that fails without this pull but succeeds with it to demonstrate the issue better ?

@jb55
Copy link
Owner

jb55 commented Jun 19, 2017

I'll just push a new major version. Those who are happy with the current behaviour can stay on 1.*

@jb55 jb55 merged commit b5aa3f3 into jb55:master Jun 19, 2017
@jb55
Copy link
Owner

jb55 commented Jun 19, 2017

available at 2.0.1

@calvinmetcalf
Copy link
Contributor Author

calvinmetcalf commented Jun 19, 2017 via email

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

Successfully merging this pull request may close these issues.

2 participants