Skip to content
This repository was archived by the owner on Jun 24, 2024. It is now read-only.

fix: Fix support for streams without content-length property#491

Merged
danielbankhead merged 1 commit intomainfrom
fix-support-for-arbitrary-length-streams
Jan 6, 2022
Merged

fix: Fix support for streams without content-length property#491
danielbankhead merged 1 commit intomainfrom
fix-support-for-arbitrary-length-streams

Conversation

@danielbankhead
Copy link
Copy Markdown
Contributor

In pumpify the prefinish event is emitted when an upstream writer has ended, not end (as with standard streams).

Fixes #490 🦕

In `pumpify` the `prefinish` event is emitted when an upstream writer has ended
@danielbankhead danielbankhead requested review from a team January 5, 2022 22:28
@product-auto-label product-auto-label Bot added the api: storage Issues related to the googleapis/gcs-resumable-upload API. label Jan 5, 2022
@danielbankhead danielbankhead added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 5, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 5, 2022
@ddelgrosso1
Copy link
Copy Markdown
Contributor

I'm assuming this is something that would have to change again if/when we ever removed pumpify?

@danielbankhead
Copy link
Copy Markdown
Contributor Author

@ddelgrosso1 correct, this is something that can be removed if pumpify is removed

@danielbankhead danielbankhead merged commit ac2f73b into main Jan 6, 2022
@danielbankhead danielbankhead deleted the fix-support-for-arbitrary-length-streams branch January 6, 2022 19:14
danielbankhead pushed a commit to googleapis/nodejs-storage that referenced this pull request Jan 6, 2022
shaffeeullah added a commit to googleapis/nodejs-storage that referenced this pull request Jan 6, 2022
* begin gcs-resumable-upload migration

* moved test

* moved system test

* unit tests passing

* fixed system tests

* 🦉 Updates from OwlBot

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

* removed log

* rename

* added index.ts

* updated headers

* temp file

* 2022 headers

* fixed buffer write

* 🦉 Updates from OwlBot

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

* changed file name

* pulled in new resumable code

* 🦉 Updates from OwlBot

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

* closed stream

* feat: Import fixes from `googleapis/gcs-resumable-upload/pull/491`

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Daniel Bankhead <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storage Issues related to the googleapis/gcs-resumable-upload API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

end to end: should work failed

4 participants