Skip remote media URLs that don't have a hostname

diff --git a/app/models/account.rb b/app/models/account.rb
index ed5c461974ad1812cba6a3597a0495e77326230c..469695acd7d8f0edd388a2dab0429d218906e7d4 100644 (file)
--- a/app/models/account.rb
+++ b/app/models/account.rb
@@ -138,7 +138,7 @@ class Account < ApplicationRecord
   def avatar_remote_url=(url)
     parsed_url = URI.parse(url)
 
-    return if !%w(http https).include?(parsed_url.scheme) || self[:avatar_remote_url] == url
+    return if !%w(http https).include?(parsed_url.scheme) || parsed_url.host.empty? || self[:avatar_remote_url] == url
 
     self.avatar              = parsed_url
     self[:avatar_remote_url] = url

diff --git a/app/services/process_feed_service.rb b/app/services/process_feed_service.rb
index f0a62aa14e84a3ee62aef5c53368c182700bc45a..5d952df6fdd2ff990bbf554240935a2f1eea14ee 100644 (file)
--- a/app/services/process_feed_service.rb
+++ b/app/services/process_feed_service.rb
@@ -181,6 +181,9 @@ class ProcessFeedService < BaseService
         next unless link['href']
 
         media = MediaAttachment.where(status: parent, remote_url: link['href']).first_or_initialize(account: parent.account, status: parent, remote_url: link['href'])
+        parsed_url = URI.parse(link['href'])
+
+        next if !%w(http https).include?(parsed_url.scheme) || parsed_url.host.empty?
 
         begin
           media.file_remote_url = link['href']