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

Add maxRedirects:0 for uploads #47

Merged
merged 1 commit into from
Oct 3, 2018

Conversation

realhidden
Copy link
Collaborator

Solves the problem of uploading large files, mentioned in #45 .

Content-Length is also passed to uploadPart the same way as in uploadFile.

Solves the problem of uploading large files
@burtontanner
Copy link

burtontanner commented Aug 31, 2018

Hey I could really use this. Could we merge this in? @yakovkhalinsky

@Bohica1
Copy link

Bohica1 commented Aug 31, 2018

This would be great to have merged in. Make life simpler

@burtontanner
Copy link

@realhidden do you know if there is an older version that would also fix this issue? I don't think it is going to get merged any time soon.

@realhidden
Copy link
Collaborator Author

You can use my dev branch: https://github.com/realhidden/backblaze-b2/tree/dev

It's easy to install in npm:

npm i git://github.com/realhidden/backblaze-b2.git#dev

Dev branch has all the 3 PRs I sent in.

@yakovkhalinsky
Copy link
Owner

@realhidden if you're interested I can give you access to this repo and add you as a contributor on the npm package.

I don't have much time these days, be a shame to waste the repo and package name 👍

@realhidden
Copy link
Collaborator Author

realhidden commented Sep 12, 2018

Sure, I would love to help.

@yakovkhalinsky
Copy link
Owner

@realhidden have added you as a collaborator, can you please email me your npm username to:
yakov at therocketsurgeon dot com

I'll then add you to the npm package so you can publish versions 👍

@realhidden realhidden merged commit 0184e51 into yakovkhalinsky:master Oct 3, 2018
@odensc odensc added this to the 1.1.0 release milestone Jan 20, 2019
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.

5 participants