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

Fixed issue where read streams on files were never ending. #850

Closed
wants to merge 1 commit into from

Conversation

nemo
Copy link

@nemo nemo commented Sep 4, 2015

Hey folks,

We were having issues with createReadStream never firing the end event.

Looks like you introduced this bug a few days ago after your recent refactor around streams. It was working fine for us until we upgraded.

Cheers.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Sep 4, 2015
@nemo
Copy link
Author

nemo commented Sep 4, 2015

Also, just signed the agreement :)

@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Sep 4, 2015
@stephenplusplus
Copy link
Contributor

Thank you for contributing! Just to confirm, you were having issues with master? Could you paste the code that we could use to reproduce? The stream should be ending itself, so it's possible the stream you're piped to isn't fully consuming the data (or a myriad of other problems!)

@nemo
Copy link
Author

nemo commented Sep 4, 2015

It may have been. It looks like you just recently made a few changes to your package.json – let me re-install and let you know if I'm facing this issue.

@nemo
Copy link
Author

nemo commented Sep 4, 2015

@stephenplusplus – looks like 0.20 fixes the issue! I was on 0.18 before.

Closing this!

@nemo nemo closed this Sep 4, 2015
@stephenplusplus
Copy link
Contributor

Great, glad it got sorted out. Please come back if you come across any new strangeness!

sofisl pushed a commit that referenced this pull request Nov 11, 2022
* docs: fix typos

PiperOrigin-RevId: 387960382

Source-Link: googleapis/googleapis@41521e2

Source-Link: googleapis/googleapis-gen@70fa757

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/master/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
sofisl pushed a commit that referenced this pull request Jan 17, 2023
…850)

Add TextDetectionParams.enable_text_detection_confidence_score to ImageContext proto so customer can optionally get confidence score for TEXT_DETECTION feature

PiperOrigin-RevId: 344839236

Source-Author: Google APIs <[email protected]>
Source-Date: Mon Nov 30 10:25:20 2020 -0800
Source-Repo: googleapis/googleapis
Source-Sha: 80a56e032bcc6a52cc41091c9a9ab527ec233f1f
Source-Link: googleapis/googleapis@80a56e0

Co-authored-by: Benjamin E. Coe <[email protected]>
sofisl pushed a commit that referenced this pull request Jan 24, 2023
Source-Link: googleapis/synthtool@d815daf
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-nodejs:latest@sha256:a9d166a74752226923d159cb723df53429e226c9c076dad3ca52ffd073ff3bb4

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
sofisl pushed a commit that referenced this pull request Jan 25, 2023
Source-Link: googleapis/synthtool@d815daf
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-nodejs:latest@sha256:a9d166a74752226923d159cb723df53429e226c9c076dad3ca52ffd073ff3bb4

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants