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 #604: Content-length greater than actual content w/ range request #609

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mintrigue
Copy link

For getAvailableSize, used when setting Content-length, I added a conditional to return the size of the actual content if it was smaller than the size of the range request.

…size is greater than full length of content
@mintrigue
Copy link
Author

issue was 604, not 607

@ghost ghost assigned thboileau Jul 18, 2012
@jlouvel jlouvel modified the milestones: 2.1.7, Unplanned issues Feb 6, 2014
@jlouvel jlouvel modified the milestones: 2.2.0, 2.1.7 Feb 7, 2014
@jlouvel
Copy link
Member

jlouvel commented Feb 7, 2014

Set milestone to 2.2.0 as it seems more work needs to be done. @thboileau can you update us?

@jlouvel jlouvel modified the milestones: 2.2 RC3, 2.2.0 Feb 20, 2014
@jlouvel
Copy link
Member

jlouvel commented Feb 20, 2014

Set milestone to 2.2 RC3

@jlouvel jlouvel modified the milestones: 2.2 RC4, 2.2 RC3, 2.2.0, 2.x - Maintenance Mar 4, 2014
@jlouvel
Copy link
Member

jlouvel commented Mar 6, 2014

Moved to 2.x maintenance.

@thboileau could you update @mintrigue with what needs to be completed on this pull request to be able to merge it.

@jlouvel jlouvel modified the milestones: 2.3 M2, 2.x - Maintenance Apr 1, 2014
@jlouvel
Copy link
Member

jlouvel commented Apr 1, 2014

Set milestone to 2.3 M2

@jlouvel jlouvel modified the milestones: 2.3 M3, 2.3 M2 Apr 30, 2014
@jlouvel jlouvel removed this from the 2.3 M3 milestone Oct 24, 2014
@jlouvel jlouvel modified the milestones: 2.3 M4, 2.3 M3, Unplanned Oct 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants