Remove the dependency on Faraday about puppet-archive HOT 7 CLOSED

I would like to switch to net::http, we can remove faraday once we implement a proper redirect:
https://github.com/lostisland/faraday_middleware/blob/master/lib/faraday_middleware/response/follow_redirects.rb

from puppet-archive.

In older issues I saw you also had ideas to remove faraday. Did you start with this?

from puppet-archive.

I started working on #55, but got hanged up because of the redirect issue.

from puppet-archive.

+1

Perhaps a simpler solution is enough:

http://stackoverflow.com/questions/6934185/ruby-net-http-following-redirects?answertab=active#tab-top

We only need to support HTTP GET, right?

from puppet-archive.

I've been looking into this. I'm guessing issue #55 is caused by the standard Net::HTTP class and not by Faraday. It looks like Net::HTTP is reading the whole response object into memory, causing memory problems when working with big files. So when we replace Faraday by our own solution, we also have to be careful not to keep the same problem.

Wouldn't it be easier to use wget. This would still introduce a dependency, but this is a rpm dependency that, most of the times is much easier. The advantage would be a battle tested en tried http/ftp and https implementation.

It supports a lot of stuff, for example:

• retries
• limit rate. This could be useful when downloading files on fragile networks
• proxies
• redirects

from puppet-archive.

This module was originally written to handle both windows/linux. A native ruby solution worked better than being dependent on a package that I could not install in windows. Both wget/curl can be implemented as providers to supplement the default provider (or even default providers on Linux). The util portion can be moved to download method so the default provider can be the parent provider with the download method replaced with your preferred solution on platforms where it's suitable.

from puppet-archive.

Sorry. Forgot about Windows for a moment ;-). I'll have my stab at making a provider for unix wget.

from puppet-archive.