search

jubalm / grunt-fontello

grunt task for downloading fonts from fontello.com
MIT License
45 stars 25 forks source link

Follow keep alive requests on fetch #33

Closed jbeezley closed 8 years ago

jbeezley commented 8 years ago

This is a parallel PR to what I submitted in https://github.com/paulyoung/fontello-cli/pull/29. The purpose is to fix frequent error messages occurring recently that look like this:

Fetching archive...Fatal error: Cannot read property 'statusCode' of undefined
brianhelba commented 8 years ago

LGTM. Merging.

brianhelba commented 8 years ago

@jubalm Can you please give me permissions to https://www.npmjs.com/package/grunt-fontello ? I'm @brianhelba there.

I'd like to issue a new patch release to fix this issue, as it's making grunt-fontello fail > 50% of the time currently.

jbeezley commented 8 years ago

As a follow up to this, I'm not 100% certain this solves the issue (or every issue) we seen. It shouldn't do any harm though.