From a860c060d5471f10201123d7340c5ffd99b9e8f0 Mon Sep 17 00:00:00 2001 From: appsu-mi Date: Fri, 22 May 2026 11:52:34 +0900 Subject: [PATCH 1/2] =?UTF-8?q?=E3=82=B9=E3=83=86=E3=83=BC=E3=82=B8?= =?UTF-8?q?=E3=83=B3=E3=82=B0=E3=81=A7=E3=81=AE=E5=95=8F=E9=A1=8C=E5=88=87?= =?UTF-8?q?=E3=82=8A=E5=88=86=E3=81=91=E3=81=AE=E3=81=9F=E3=82=81=E6=80=AA?= =?UTF-8?q?=E3=81=97=E3=81=84=E3=81=A8=E3=81=93=E3=82=8D=E3=81=AB=E3=83=AD?= =?UTF-8?q?=E3=82=B0=E5=87=BA=E5=8A=9B=E3=82=92=E5=9F=8B=E3=82=81=E8=BE=BC?= =?UTF-8?q?=E3=82=80?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/models/link_checker/checker.rb | 3 ++- app/models/metadata.rb | 12 ++++++++++-- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/app/models/link_checker/checker.rb b/app/models/link_checker/checker.rb index 2737b9bbf21..f5b0a94e2e5 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}") false end diff --git a/app/models/metadata.rb b/app/models/metadata.rb index 53a9ef7fa37..ae3fc60ff65 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}") + 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}") + return + end { title: object.og.title, From cfe4eae485ce27b652ac41b1b81ecf56758f3938 Mon Sep 17 00:00:00 2001 From: appsu-mi Date: Fri, 22 May 2026 12:08:53 +0900 Subject: [PATCH 2/2] =?UTF-8?q?=E6=9C=AC=E7=95=AA=E7=92=B0=E5=A2=83?= =?UTF-8?q?=E3=81=AB=E6=BC=8F=E3=82=8C=E3=81=AA=E3=81=84=E3=82=88=E3=81=86?= =?UTF-8?q?=E3=81=AB=E3=81=99=E3=82=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/models/link_checker/checker.rb | 2 +- app/models/metadata.rb | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/models/link_checker/checker.rb b/app/models/link_checker/checker.rb index f5b0a94e2e5..4f8396c1741 100644 --- a/app/models/link_checker/checker.rb +++ b/app/models/link_checker/checker.rb @@ -40,7 +40,7 @@ def valid_domain?(domain) Resolv.getaddress(domain) true rescue Resolv::ResolvError => e - Rails.logger.info("[LinkChecker] #{e.class}: domain=#{domain}") + 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 ae3fc60ff65..a713dfa23f8 100644 --- a/app/models/metadata.rb +++ b/app/models/metadata.rb @@ -16,7 +16,7 @@ def fetch response = http.request_get(@uri.request_uri) if response.message != 'OK' - Rails.logger.info("[Metadata] Response was not OK: status=#{response.code} url=#{@url}") + Rails.logger.info("[Metadata] Response was not OK: status=#{response.code} url=#{@url}") unless Rails.env.production? return end @@ -28,7 +28,7 @@ def fetch def parse(html) object = OpenGraphReader.parse(html) unless object - Rails.logger.info("[Metadata] OpenGraphReader parse failed: url=#{@url}, title=#{Nokogiri::HTML(html).title}") + Rails.logger.info("[Metadata] OpenGraphReader parse failed: url=#{@url}, title=#{Nokogiri::HTML(html).title}") unless Rails.env.production? return end