diff --git a/app/models/link_checker/checker.rb b/app/models/link_checker/checker.rb index 2737b9bbf21..4f8396c1741 100644 --- a/app/models/link_checker/checker.rb +++ b/app/models/link_checker/checker.rb @@ -39,7 +39,8 @@ def valid_domain?(domain) Resolv.getaddress(domain) true - rescue Resolv::ResolvError + rescue Resolv::ResolvError => e + Rails.logger.info("[LinkChecker] #{e.class}: domain=#{domain}") unless Rails.env.production? false end diff --git a/app/models/metadata.rb b/app/models/metadata.rb index 53a9ef7fa37..a713dfa23f8 100644 --- a/app/models/metadata.rb +++ b/app/models/metadata.rb @@ -15,14 +15,22 @@ def fetch http.response_body_encoding = true response = http.request_get(@uri.request_uri) - response.message == 'OK' ? parse(response.body) : nil + if response.message != 'OK' + Rails.logger.info("[Metadata] Response was not OK: status=#{response.code} url=#{@url}") unless Rails.env.production? + return + end + + parse(response.body) end private def parse(html) object = OpenGraphReader.parse(html) - return unless object + unless object + Rails.logger.info("[Metadata] OpenGraphReader parse failed: url=#{@url}, title=#{Nokogiri::HTML(html).title}") unless Rails.env.production? + return + end { title: object.og.title,