Allow NetworkLoader to be configured with support for proxied requests

[plasticine]

Allow NetworkLoader to be configured with support for proxied requests

[daniel-rikowski]

I have a similar monkey patch for ignoring a self-signed certificate in development.

Perhaps it would be better to use a "downloader proc" instead of providing individual parameters.

Something like this:

def load(url)
  uri = uri_for_url(url) || return
  downloader = Rails.config[:network_downloader] || ->(uri) { Net::HTTP.get(uri) }
  downloader.call(uri)
end

Premailer::Rails.config.merge!(
  # With SSL options:
  network_downloader: lambda do |uri|
    request = Net::HTTP::Get.new(uri.path)
    response = Net::HTTP.start(
        uri.host, uri.port, 
        use_ssl: uri.scheme == 'https', 
        verify_mode: OpenSSL::SSL::VERIFY_NONE) do |https|
      https.request(request)
    end
    response.body
  end,

  # With proxy:
  network_downloader: lambda do |uri|
    request = Net::HTTP.new(uri.host, uri.port, ENV['egress_proxy_host'], ENV['egress_proxy_port'])
    request.use_ssl = uri.scheme == 'https'
    response = request.get(uri.request_uri)
    response.body  
  end
)

[fphilipe]

Not going to support this. But you have a working solution 😊 🐒🩹