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 support for getting only portions of objects via HTTP range requests #58

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

greenlion
Copy link

I made getObject() take a forth parameter. Now the third or fourth parameter may be an array (the forth is an array if the third is a filename or resource). If an array is passed in, then it should have two indices [0] and [1] which point to start and end bytes, respectively.

ie:
$first_ten_bytes = S3::getObject($bucketName, $uploadName, array(1,10));

Of course, if no range is passed in, then the whole object is retrieved.

@tpyo tpyo mentioned this pull request Oct 5, 2013
@greenlion
Copy link
Author

sorry it is a bit of a dirty push. I was in a hurry. I can clean it up and make any changes you request if you like. I didn't see issue #10, I just wrote this for myself and pushed it.

@greenlion
Copy link
Author

did you add this functionality? if not, can you pull it as is. it has languished. if you don't want to pull it, what changes to do you want?

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.

1 participant