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

Skip progress report when Content-Length is unavailable #86

Merged

Conversation

eagletmt
Copy link
Contributor

@eagletmt eagletmt commented Sep 3, 2018

This should fix #85

@Paxa
Copy link

Paxa commented Sep 7, 2018

Thanks for fixing it!

@thijsc
Copy link

thijsc commented Sep 21, 2018

Anything holding this back? Would be nice to get this out in a release, it would fix installation of the rdkafka gem. Probably effects a lot of other gems using the same approach too.

@flavorjones
Copy link
Owner

Sorry, nothing is holding this back except my inability to find time to work on OSS this month. Will get a release out this weekend.

@flavorjones flavorjones self-assigned this Sep 22, 2018
steventux pushed a commit to alphagov/govuk-content-schemas that referenced this pull request Oct 11, 2018
Miniportile is used to retrieve jsonnet from github.
The progress reporter in miniportile can't handle a nil
content-length response which github sends with content-
encoding: Chunked.
There's a fix flavorjones/mini_portile#86
which needs merging into the gem at which point we can
remove this env var.
@kevindew
Copy link

kevindew commented Nov 5, 2018

@flavorjones Any chance you'll get to look at this again soon?

@flavorjones flavorjones merged commit ce0e270 into flavorjones:master Dec 2, 2018
@flavorjones
Copy link
Owner

Hey all -- apologies for the slow response. I've merged this! Will cut a release shortly.

@flavorjones
Copy link
Owner

mini_portile2 v2.4.0 has been shipped. Thanks again!

kevindew added a commit to alphagov/govuk-content-schemas that referenced this pull request Dec 6, 2018
This updates our gems most noticably mini_portile2 which has had an
issue making it difficult to install jsonnet. See
flavorjones/mini_portile#86 for more details.
@eagletmt eagletmt deleted the skip-progress-when-chunked branch December 11, 2018 04:38
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.

Unable to download files when Content-Encoding is chunked
5 participants