directlink 0.0.9.1 → 0.0.11.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: '08e29b8a3032310a9443ca5f2bc001c449722aa8'
-  data.tar.gz: 15ee2b72e8ea566e2250a0311afe530a5b9dbd36
+  metadata.gz: e0cadc1fae7bcad2314224fa404ca05dff9988e6
+  data.tar.gz: e5c4c32ac6e03efa4633b9dc323c8a44d8f52c4e
 SHA512:
-  metadata.gz: b30d00607103ed669f5876b9360b8280e24a6ba66c06117f577e47b497eb1ed76eee953bf0cfc97a2525dffc7c0f1169377cbcb780d7d2c9c1a1d74e9aaec212
-  data.tar.gz: 36a8ecce05a750e1b5aea43929037d67c38f4dcc0df8d022739aa7508a01d51d573e3f39f0ebbca14fca2dbb90d29eb2be6ae74949b77e1e735da7c2be16ac04
+  metadata.gz: c96f80c6d75a232d282112cc2aad8a085e7c324df5bce7951cde142a11ec94346a53ce2a1f34ad073215307220540402ea64442ea9f55ee9ad2910aa9eac349e
+  data.tar.gz: 00f67755c1df4fab9aa7f2819b7b633f3d2462e3b55b767f38d5d3e458f20ce50feb45e50735d7fd06900d97bb2201a80425aa3852a82c569cebfe40fcedd6ca

data/bin/directlink

@@ -68,18 +68,13 @@ begin
       (t.is_a?(Array) ? t : [t]).each{ |s| puts "=> #{s.url}\n   #{s.type} #{s.width}x#{s.height}" }
     end
   end
-rescue SocketError,
-       Net::OpenTimeout,
-       Errno::ECONNRESET,
-       NetHTTPUtils::Error,
-       FastImage::UnknownImageType,
-       FastImage::ImageFetchFailure,
-       # DirectLink::ErrorMissingEnvVar,
-       # DirectLink::ErrorAssert,
-       DirectLink::ErrorNotFound,
-       DirectLink::ErrorBadLink => e
+rescue *DirectLink::NORMAL_EXCEPTIONS => e
   puts e.backtrace if debug
   cause = e.cause if e.cause if e.respond_to? :cause
   c = e.class.to_s
   abort "#{c}#{": #{e}" if c != e.to_s}#{": #{cause}" if cause && c != cause.to_s}"
+rescue
+  raise                unless $!.cause
+  raise $!.cause       unless $!.cause.cause
+  raise $!.cause.cause
 end

data/directlink.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |spec|
   spec.name         = "directlink"
-  spec.version      = "0.0.9.1"
-  spec.summary      = "converts any kind of image hyperlink to direct link, type of image and its resolution"
+  spec.version      = "0.0.11.1"
+  spec.summary      = "obtains from any kind of hyperlink a link to an image, its format and resolution"
 
   spec.author       = "Victor Maslov aka Nakilon"
   spec.email        = "nakilon@gmail.com"
@@ -9,21 +9,23 @@ Gem::Specification.new do |spec|
   spec.homepage     = "https://github.com/nakilon/directlink"
   spec.metadata     = {"source_code_uri" => "https://github.com/nakilon/directlink"}
 
-  spec.add_dependency "fastimage", "~>2.1.3"
-  spec.add_dependency "nokogiri"
-  spec.add_dependency "nethttputils", "~>0.4.0.0"
-  spec.add_dependency "reddit_bot", "~>1.7.0"
+  spec.required_ruby_version = ">=2.3"  # because <<~ heredocs in tests
+
+  spec.add_dependency "fastimage", "~>2.2.0"
+  spec.add_dependency "nokogiri", "<1.11"   # 1.11 requires ruby 2.5  # TODO: switch to Oga?
+  # spec.add_dependency "nethttputils", "~>0.4.2.0"
+  spec.add_dependency "reddit_bot", "~>1.10.0"
   spec.add_dependency "kramdown"
   spec.add_dependency "addressable"
-  spec.add_development_dependency "minitest"
-  spec.add_development_dependency "byebug"
+  spec.add_development_dependency "minitest-around"
+  spec.add_development_dependency "webmock"
 
   spec.require_path = "lib"
   spec.bindir       = "bin"
   spec.executable   = "directlink"
-  spec.test_file    = "test.rb"
-  spec.files        = `git ls-files -z`.split(?\0) - spec.test_files
+  spec.test_file    = "unit.test.rb"
+  spec.files        = %w{ LICENSE directlink.gemspec lib/directlink.rb bin/directlink }
 
-  spec.requirements << "you may need to create apps and provide their API tokens:"
+  spec.requirements << "you may want to create apps and provide API tokens:"
   spec.requirements << "IMGUR_CLIENT_ID, FLICKR_API_KEY, REDDIT_SECRETS"
 end

data/lib/directlink.rb

@@ -2,17 +2,16 @@ module DirectLink
 
   class << self
     attr_accessor :silent
-  end
-  self.silent = false
-  class << self
     attr_accessor :logger
+    attr_accessor :timeout
   end
+  self.silent = false
   self.logger = Object.new
   self.logger.define_singleton_method :error do |str|
     puts str unless Module.nesting.first.silent
   end
 
-  class ErrorAssert < RuntimeError
+  class ErrorAssert < RuntimeError  # gem user should not face this error
     def initialize msg
       super "#{msg} -- consider reporting this issue to GitHub"
     end
@@ -35,6 +34,22 @@ module DirectLink
     end
   end
 
+  require "nethttputils"
+  require "fastimage"
+  NORMAL_EXCEPTIONS = [
+    SocketError,
+    Net::OpenTimeout,
+    Errno::ECONNRESET,
+    Errno::ECONNREFUSED,
+    Errno::ETIMEDOUT,   # from FastImage
+    NetHTTPUtils::Error,
+    NetHTTPUtils::EOFError_from_rbuf_fill,
+    FastImage::UnknownImageType,
+    FastImage::ImageFetchFailure,
+    DirectLink::ErrorNotFound,
+    DirectLink::ErrorBadLink,
+  ]  # all known exceptions that can be raised while using Directlink but not as its fault
+
 
   def self.google src, width = 0
     # this can handle links without schema because it's used for parsing community HTML pages
@@ -44,7 +59,7 @@ module DirectLink
       "#{$1}s#{width}/"
     when /\A(\/\/lh3\.googleusercontent\.com\/proxy\/[a-zA-Z0-9_-]{66,523}=)(?:w(?:[45]\d\d)-h\d\d\d-[np]|s530-p|s110-p-k)\z/
       "https:#{$1}s#{width}/"
-    when /\A(\/\/lh3\.googleusercontent\.com\/cOh2Nsv7EGo0QbuoKxoKZVZO_NcBzufuvPtzirMJfPmAzCzMtnEncfA7zGIDTJfkc1YZFX2MhgKnjA=)w530-h398-p\z/
+    when /\A(\/\/lh3\.googleusercontent\.com\/[a-zA-Z0-9]{24}_[a-zA-Z]{30}7zGIDTJfkc1YZFX2MhgKnjA=)w530-h398-p\z/
       "https:#{$1}s#{width}/"
     when /\A(\/\/lh3\.googleusercontent\.com\/-[a-zA-Z0-9-]{11}\/[VW][a-zA-Z0-9_-]{9}I\/AAAAAAA[AC][a-zA-Z0-9]{3}\/[a-zA-Z0-9_-]{32}[gwAQ]CJoC\/)w530-h[23]\d\d-p\/[^\/]+\z/,
          /\A(?:https?:)?(\/\/[1-4]\.bp\.blogspot\.com\/-[a-zA-Z0-9_-]{11}\/[UVWX][a-zA-Z0-9_-]{9}I\/AAAAAAAA[A-Z][a-zA-Z0-9_-]{2}\/[a-zA-Z0-9_-]{33}C(?:EwYBhgL|(?:Lc|Kg)BGAs(?:YHQ)?)\/)(?:s640|w\d{2,4}-h\d\d\d?-p(?:-k-no-nu)?)\/[^\/]+\z/,
@@ -65,18 +80,24 @@ module DirectLink
          /\A(https:\/\/lh[356]\.googleusercontent\.com\/-[a-zA-Z0-9]{11}\/AAAAAAAAAAI\/AAAAAAAAAAA\/[a-zA-Z0-9_]{34}\/)s(?:46|64)-c(?:-k(?:-no)?)?-mo\/photo\.jpg\z/
       "#{$1}s#{width}/"
     # Google Keep
-    when /\A(https:\/\/lh\d\.googleusercontent\.com\/[a-zA-Z0-9_-]{104,106}=s)\d\d\d\d?\Z/
+    when /\A(https:\/\/lh\d\.googleusercontent\.com\/[a-zA-Z0-9_-]{104,106}=s)\d\d\d\d?\z/
       "#{$1}#{width}"
+    # opensea
+    when /\A(https:\/\/lh3\.googleusercontent\.com\/[a-zA-Z0-9]{78}-nGx_jf_XGqqiVANe_Jr8u2g=)w1400-k\z/
+      "#{$1}s#{width}"
     # mp4
     when /\A(https:\/\/lh3\.googleusercontent\.com\/-[a-zA-Z]{11}\/W[a-zA-Z0-9]{9}I\/AAAAAAAAODw\/[a-zA-Z0-9]{32}QCJoC\/)w530-h883-n-k-no\/[^\/]+\.mp4\z/
       "#{$1}s#{width}/"
+    # something else
+    when /\A(https:\/\/lh3\.googleusercontent\.com\/-[a-zA-Z0-9_]{11}\/X-[a-zA-Z0-9]{8}I\/AAAAAAAAALE\/[a-zA-Z0-9]{23}_[a-zA-Z0-9]{19}\/)w1200-h630-p-k-no-nu\/[\d-]+\.png\z/
+      "#{$1}s#{width}/"
     else
       raise ErrorBadLink.new src
     end
   end
 
+
   require "json"
-  require "nethttputils"
 
   # TODO make the timeout handling respect the way the Directlink method works with timeouts
   def self.imgur link, timeout = 1000
@@ -88,7 +109,7 @@ module DirectLink
         NetHTTPUtils.request_data url, header: { Authorization: "Client-ID #{ENV["IMGUR_CLIENT_ID"]}" }
       rescue NetHTTPUtils::Error => e
         raise ErrorNotFound.new url.inspect if 404 == e.code
-        if t < timeout && [400, 500, 503].include?(e.code)
+        if t < timeout && [400, 500, 502, 503].include?(e.code)
           logger.error "retrying in #{t} seconds because of Imgur HTTP ERROR #{e.code}"
           sleep t
           t *= 2
@@ -107,16 +128,16 @@ module DirectLink
       elsif data["images"]
         raise ErrorNotFound.new link.inspect if data["images"].empty?
         data["images"]
-      elsif data["type"] && data["type"].start_with?("image/")
+      elsif data["type"] && %w{ image/jpeg image/png image/gif video/mp4 }.include?(data["type"])
         # TODO check if this branch is possible at all
         [ data ]
       # elsif data["comment"]
       #   fi["https://imgur.com/" + data["image_id"]]
       else
         # one day single-video item should hit this but somehow it didn't yet
-        raise ErrorAssert.new "unknown data format #{data.inspect} for #{link}"
+        raise ErrorAssert.new "unknown data format #{json} for #{link}"
       end
-    when /\Ahttps?:\/\/(?:(?:i|m|www)\.)?imgur\.com\/([a-zA-Z0-9]{7,8})(?:\.(?:gifv|jpg(?:\?fb)?|png))?\z/,
+    when /\Ahttps?:\/\/(?:(?:i|m|www)\.)?imgur\.com\/([a-zA-Z0-9]{7,8})(?:\.(?:gifv|jpe?g(?:\?fb)?|png))?\z/,
          /\Ahttps?:\/\/(?:(?:i|m|www)\.)?imgur\.com\/([a-zA-Z0-9]{5})\.mp4\z/,
          /\Ahttps?:\/\/imgur\.com\/([a-zA-Z0-9]{5}(?:[a-zA-Z0-9]{2})?)\z/,
          /\Ahttps?:\/\/imgur\.com\/([a-zA-Z0-9]{7})(?:\?\S+)?\z/,
@@ -128,7 +149,7 @@ module DirectLink
       raise ErrorBadLink.new link
     end.map do |image|
       case image["type"]
-      when "image/jpeg", "image/png", "image/gif", "video/mp4"
+      when *%w{ image/jpeg image/png image/gif video/mp4 }
         image.values_at "link", "width", "height", "type"
       else
         raise ErrorAssert.new "unknown type of #{link}: #{image}"
@@ -139,7 +160,6 @@ module DirectLink
   def self._500px link
     raise ErrorBadLink.new link unless %r{\Ahttps://500px\.com/photo/(?<id>[^/]+)/[-[a-zA-Z0-9]%]+\/?\z} =~ link
     require "nokogiri"
-    resp = NetHTTPUtils.request_data link
     f = lambda do |form|
       JSON.load(NetHTTPUtils.request_data "https://api.500px.com/v1/photos", form: form).fetch("photos").values.first
     end
@@ -191,11 +211,11 @@ module DirectLink
     attr_accessor :reddit_bot
   end
   def self.reddit link, timeout = 1000
-    unless id = URI(link).path[/\A(?:\/r\/[0-9a-zA-Z_]+)?(?:\/comments|\/duplicates)?\/([0-9a-z]{5,6})(?:\/|\z)/, 1]
-      raise DirectLink::ErrorBadLink.new link unless URI(link).host &&
-                                                     URI(link).host.split(?.) == %w{ i redd it } &&
-                                                     URI(link).path[/\A\/[a-z0-9]{12,13}\.(gif|jpg)\z/]
-      return [true, link]
+    return [true, link] if URI(link).host &&
+                           URI(link).host.split(?.) == %w{ i redd it } &&
+                           URI(link).path[/\A\/[a-z0-9]{12,13}\.(gif|jpg)\z/]
+    unless id = link[/\Ahttps:\/\/www\.reddit\.com\/gallery\/([0-9a-z]{5,6})\z/, 1]
+      raise DirectLink::ErrorBadLink.new link unless id = URI(link).path[/\A(?:\/r\/[0-9a-zA-Z_]+)?(?:\/comments|\/duplicates)?\/([0-9a-z]{5,6})(?:\/|\z)/, 1]
     end
     retry_on_json_parseerror = lambda do |&b|
       t = 1
@@ -223,17 +243,20 @@ module DirectLink
     end
     # TODO: do we handle linking Imgur albums?
     data = json["data"]["children"].first["data"]
-    if data["media"]["reddit_video"]
-      return [true, data["media"]["reddit_video"]["fallback_url"]]
-    else
+    if data["media"]
+      return [true, data["media"]["reddit_video"]["fallback_url"]] if data["media"]["reddit_video"]
       raise ErrorAssert.new "our knowledge about Reddit API seems to be outdated" unless data["media"].keys.sort == %w{ oembed type } && %w{ youtube.com gfycat.com imgur.com }.include?(data["media"]["type"])
       return [true, data["media"]["oembed"]["thumbnail_url"]]
-    end if data["media"]
-    return [true, data["media_metadata"].values.map do |media|
-      [media["m"], *media["p"].max_by{ |_| _["x"] * _["y"] }.values_at("x", "y", "u")]
-    end] if data["media_metadata"]
-    return [true, data["url"]] if data["crosspost_parent"]
-    return [true, data["url"]] unless data["is_self"]
+    end
+    if data["media_metadata"]
+      return [true, data["media_metadata"].values.map do |media|
+        next if media == {"status"=>"failed"} || media == {"status"=>"unprocessed"}
+        raise ErrorAssert.new "our knowledge about Reddit API seems to be outdated (media == #{media.inspect})" unless media["status"] == "valid"
+        [media["m"], *media["s"].values_at("x", "y"), CGI.unescapeHTML(media["s"][media["m"]=="image/gif" ? "gif" : "u"])]
+      end.compact]
+    end
+    return [true, "#{"https://www.reddit.com" if /\A\/r\/[0-9a-zA-Z_]+\/comments\/[0-9a-z]{5,6}\// =~ data["url"]}#{data["url"]}"] if data["crosspost_parent"]
+    return [true, CGI.unescapeHTML(data["url"])] unless data["is_self"]
     raise ErrorAssert.new "our knowledge about Reddit API seems to be outdated" if data["url"] != "https://www.reddit.com" + data["permalink"]
     return [false, data["selftext"]]
   end
@@ -242,7 +265,7 @@ module DirectLink
     id, mtd, field, f = case link
     when %r{\Ahttps://vk\.com/id(?<user_id>\d+)\?z=photo(?<id>\k<user_id>_\d+)(%2F(album\k<user_id>_0|photos\k<user_id>))?\z},
          %r{\Ahttps://vk\.com/[a-z_]+\?z=photo(?<_>)(?<id>(?<user_id>\d+)_\d+)%2Fphotos\k<user_id>\z},
-         %r{\Ahttps://vk\.com/photo(?<_>)(?<id>-?\d+_\d+)(\?all=1)?\z},
+         %r{\Ahttps://vk\.com/photo(?<_>)(?<id>-?\d+_\d+)(\?(?:all|rev)=1)?\z},
          %r{\Ahttps://vk\.com/feed\?section=likes&z=photo(?<_>)(?<id>-(?<user_id>\d+)_\d+)%2F(liked\d+|album\k<user_id>_0)\z},
          %r{\Ahttps://vk\.com/[a-z_]+\?z=photo(?<_>)(?<id>(?<user_id>-\d+)_\d+)%2F(wall\k<user_id>_\d+|album\k<user_id>_0)\z},
          %r{\Ahttps://vk\.com/wall(?<user_id>-\d+)_\d+\?z=photo(?<id>\k<user_id>_\d+)%2F(wall\k<user_id>_\d+|album\k<user_id>_00%2Frev|\d+)\z}
@@ -251,7 +274,7 @@ module DirectLink
         t
       end ]
     when %r{\Ahttps://vk\.com/wall(?<id>-?\d+_\d+)\z},
-         %r{\Ahttps://vk\.com/[a-z\.]+\?w=wall(?<id>\d+_\d+)\z}
+         %r{\Ahttps://vk\.com/[a-z\.]+\?w=wall(?<id>-?\d+_\d+)\z}
       [$1, :wall, :posts, lambda do |t|
         t.first.fetch("attachments").select do |item|
           case item.keys
@@ -269,14 +292,14 @@ module DirectLink
       raise ErrorBadLink.new link
     end
     raise ErrorMissingEnvVar.new "define VK_ACCESS_TOKEN and VK_CLIENT_SECRET env vars" unless ENV["VK_ACCESS_TOKEN"] && ENV["VK_CLIENT_SECRET"]
-    sleep 0.25 # "error_msg"=>"Too many requests per second"
+    sleep 0.25 unless ENV["CI"] # "error_msg"=>"Too many requests per second"
     f.call( JSON.load( NetHTTPUtils.request_data "https://api.vk.com/method/#{mtd}.getById",
       :POST, form: { field => id, :access_token => ENV["VK_ACCESS_TOKEN"], :client_secret => ENV["VK_CLIENT_SECRET"], :v => "5.101" }
     ).fetch("response") ).map do |photos|
       photos.fetch("sizes").map do |size|
         size.values_at("width", "height", "url").tap do |whu|
           w, h, u = whu
-          whu[0, 2] = FastImage.new(u, raise_on_failure: true).size if [w, h].include? 0
+          whu[0, 2] = FastImage.new(u, raise_on_failure: true).size if [w, h].include? 0 # wtf?
         end
       end.max_by{ |w, h, u| w * h }
     end
@@ -286,9 +309,8 @@ module DirectLink
 end
 
 
-require "fastimage"
-
-def DirectLink link, timeout = nil, giveup: false, ignore_meta: false
+def DirectLink link, timeout = nil, proxy = nil, giveup: false, ignore_meta: false
+  timeout ||= DirectLink.timeout
   ArgumentError.new("link should be a <String>, not <#{link.class}>") unless link.is_a? String
   begin
     URI link
@@ -326,15 +348,16 @@ def DirectLink link, timeout = nil, giveup: false, ignore_meta: false
       **( %w{ reddit com } == URI(link).host.split(?.).last(2) ||
           %w{   redd it  } == URI(link).host.split(?.) ? {Cookie: "over18=1"} : {} ),
     }
-    head = NetHTTPUtils.request_data link, :head, header: header, **(timeout ? {
+    head = NetHTTPUtils.request_data link, :HEAD, header: header, **(proxy ? {proxy: proxy} : {}), **(timeout ? {
       timeout: timeout,
       max_start_http_retry_delay: timeout,
-      max_read_retry_delay: timeout
+      max_read_retry_delay: timeout,
     } : {})
-  rescue Net::ReadTimeout
+  rescue Net::ReadTimeout, Errno::ETIMEDOUT
   rescue NetHTTPUtils::Error => e
     raise unless 418 == e.code
   else
+    raise DirectLink::ErrorAssert.new "last_response.uri is not set" unless head.instance_variable_get(:@last_response).uri
     link = head.instance_variable_get(:@last_response).uri.to_s
   end
 
@@ -342,10 +365,10 @@ def DirectLink link, timeout = nil, giveup: false, ignore_meta: false
   #   because they can be hidden behind URL shorteners
   #   also it can resolve NetHTTPUtils::Error(404) before trying the adapter
 
-  t = google_without_schema_crutch[] and return t
+  t = google_without_schema_crutch[] and return t   # TODO: why again?
 
   begin
-    imgur = DirectLink.imgur(link).sort_by{ |u, w, h, t| - w * h }.map do |u, w, h, t|
+    imgur = DirectLink.imgur(link, timeout).sort_by{ |u, w, h, t| - w * h }.map do |u, w, h, t|
       struct.new u, w, h, t
     end
     # `DirectLink.imgur` return value is always an Array
@@ -383,7 +406,7 @@ def DirectLink link, timeout = nil, giveup: false, ignore_meta: false
       f = ->_{ _.type == :a ? _.attr["href"] : _.children.flat_map(&f) }
       require "kramdown"
       return f[Kramdown::Document.new(u).root].flat_map do |sublink|
-        DirectLink URI.join(link, sublink).to_s, timeout, giveup: giveup
+        DirectLink URI.join(link, sublink).to_s, timeout, giveup: giveup   # TODO: maybe subtract from timeout the time we've already wasted
       end
     end
     if u.is_a? Hash
@@ -393,8 +416,8 @@ def DirectLink link, timeout = nil, giveup: false, ignore_meta: false
         struct.new u, x, y, t
       end
     end
-    return DirectLink u
-    fail if link == u
+    raise DirectLink::ErrorNotFound.new link.inspect if link == u
+    return DirectLink u, timeout, giveup: giveup
   rescue DirectLink::ErrorMissingEnvVar
   end if %w{ reddit com } == URI(link).host.split(?.).last(2) ||
          %w{   redd it  } == URI(link).host.split(?.)
@@ -407,23 +430,27 @@ def DirectLink link, timeout = nil, giveup: false, ignore_meta: false
   end if %w{ vk com } == URI(link).host.split(?.)
 
   begin
-    f = FastImage.new(link, raise_on_failure: true, timeout: 5, http_header: {"User-Agent" => "Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.139 Safari/537.36"})
+    f = FastImage.new link,
+      raise_on_failure: true,
+      timeout: timeout,
+      **(proxy ? {proxy: "http://#{proxy}"} : {}),
+      http_header: {"User-Agent" => "Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.139 Safari/537.36"}
   rescue FastImage::UnknownImageType
     raise if giveup
     require "nokogiri"
-    head = NetHTTPUtils.request_data link, :head, header: {"User-Agent" => "Mozilla"},
+    head = NetHTTPUtils.request_data link, :HEAD, header: {"User-Agent" => "Mozilla"},
       max_start_http_retry_delay: timeout,
       timeout: timeout,                 # NetHTTPUtild passes this as read_timeout to Net::HTTP.start
       max_read_retry_delay: timeout     # and then compares accumulated delay to this
     # if we use :get here we will download megabytes of files just to giveup on content_type we can't process
-    case head.instance_variable_get(:@last_response).content_type
+    case head.instance_variable_get(:@last_response).content_type   # webmock should provide this
     when "text/html" ; nil
     else ; raise
     end
-    html = Nokogiri::HTML NetHTTPUtils.request_data link, header: {"User-Agent" => "Mozilla"}
+    html = Nokogiri::HTML NetHTTPUtils.request_data link, :GET, header: {"User-Agent" => "Mozilla"}
     if t = html.at_css("meta[@property='og:image']")
       begin
-        return DirectLink URI.join(link, t[:content]).to_s, nil, giveup: true
+        return DirectLink URI.join(link, t[:content]).to_s, nil, *proxy, giveup: true
       rescue URI::InvalidURIError
       end
     end unless ignore_meta