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

Support for Gzip/Inflate #113

Closed
ixti opened this issue Mar 25, 2014 · 5 comments · Fixed by #375
Closed

Support for Gzip/Inflate #113

ixti opened this issue Mar 25, 2014 · 5 comments · Fixed by #375

Comments

@ixti
Copy link
Member

ixti commented Mar 25, 2014

Would like to see HTTP gem playing well with GZip/Inflate out of the box.

@ixti ixti self-assigned this Apr 5, 2014
@sferik sferik added this to the v0.7 milestone Apr 19, 2014
@ixti ixti removed this from the v0.7 milestone May 26, 2014
@sferik sferik modified the milestone: v0.8.0 Mar 28, 2015
@ixti ixti modified the milestones: v0.8, v1.0 Mar 29, 2015
@ixti
Copy link
Member Author

ixti commented Mar 29, 2015

@sferik I doubt we will be able to get this support on 0.8.0 :D I have moved this to 1.0.0, if we will be able to sneak this in 0.9.0 - awesome, if not - well, 1.0.0 still seems reasonable to me :D

@tarcieri tarcieri modified the milestones: v0.8, v1.0 Mar 29, 2015
@ixti ixti added the Pick Me! label Mar 30, 2015
@sferik sferik modified the milestones: v0.9, v1.0 Mar 30, 2015
@sferik
Copy link
Contributor

sferik commented Mar 30, 2015

I moved this into the v0.9 milestone, since there isn’t much else in there, so I’m optimistic that this can be implemented ahead of the v1.0 release. I view the entire v0.9 series as a pre-release of v1.0. In theory, we shouldn’t be making any breaking changes between v0.9 and v1.0.

If we release v0.9 and it’s stable for a month, we can ship it as v1.0.

@zanker
Copy link
Contributor

zanker commented Mar 31, 2015

I can't remember if this is extremely hard or extremely easy, I think the streaming makes it more annoying but I can double check on doing this quickly.

@tarcieri
Copy link
Member

It's pretty easy to do with Zlib::Inflate

@zanker
Copy link
Contributor

zanker commented Mar 31, 2015

I thought it was supposed to be more annoying to stream but apparently I was wrong. Maybe I'm thinking of something else

/shrug

I'll take a look into it for 0.9 then

@ixti ixti removed their assignment Apr 12, 2015
@ixti ixti self-assigned this May 24, 2015
@ixti ixti modified the milestones: v1.1.0, v2.0.0 Nov 1, 2015
@ixti ixti removed this from the v1.1 milestone Mar 20, 2016
@ixti ixti modified the milestones: v2.0, v1.1 Mar 20, 2016
@ixti ixti removed their assignment Mar 20, 2016
@ixti ixti modified the milestones: v2.0, v2.x, v2.1, v3.0 May 5, 2016
Bonias pushed a commit to Bonias/http that referenced this issue Sep 20, 2016
@ixti ixti closed this as completed in #375 Dec 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants