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

Allow storage write streams to start at an offset #1325

Closed
wants to merge 1 commit into from

Conversation

bhstahl
Copy link

@bhstahl bhstahl commented May 17, 2016

The gcs-resumable-upload package was updated to allow a users to specify a starting offset by making the Content-Range header configurable.

This PR allows the offset to be passed through in Bucket#upload and File#createWriteStream.

Should be merged after #1322, as that PR bumps gcs-resumable-upload to the new version.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 17, 2016
@stephenplusplus
Copy link
Contributor

Merged in b877420. Thanks again!

@bhstahl
Copy link
Author

bhstahl commented May 18, 2016

Thanks @stephenplusplus ! Are you planning to do a release and/or bump the npm version? I'm looking to get these changes in tus-node-server, but I can install from my fork if necessary

@stephenplusplus
Copy link
Contributor

There's one more PR I'd like to land first (#1320), but if that ends up needing more time, I'll do a release without it. Basically, I'll try to get a release out before 5:00 (EDT) today 🕔

@bhstahl
Copy link
Author

bhstahl commented May 18, 2016

Sounds great, thanks. I'm happy to help out too if you need anything in particular

@stephenplusplus
Copy link
Contributor

Thanks :) Just an update, I just opened a PR for a last minute bug report that came in. I'd like to make sure that works for the reporter, then it's release time.

@stephenplusplus
Copy link
Contributor

Phew. 0.34.0 has landed.

@bhstahl
Copy link
Author

bhstahl commented May 18, 2016

Awesome!

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