Embedding está QUEBRADO em 3.0.4 (estável)

Bug
7

Então main tem isso

TopicEmbed.import_remote(@embed_url, user: User.find_by(username_lower: username.downcase))

e stable tem isso

TopicEmbed.import_remote(user, @embed_url)

Note a ordem dos parâmetros.

Agora o backport do patch de segurança mudou a assinatura da função no stable para a nova ordem de parâmetros, então

def self.import_remote(import_user, url, opts = nil)

tornou-se

def self.import_remote(url, opts = nil)

e agora o parâmetro url recebe um objeto User.

Mudar a chamada da função resolve o problema

diff --git a/lib/topic_retriever.rb b/lib/topic_retriever.rb
index b798df6cd7..6186ce5868 100644
--- a/lib/topic_retriever.rb
+++ b/lib/topic_retriever.rb
@@ -50,6 +50,6 @@ class TopicRetriever
     user = User.where(username_lower: username.downcase).first
     return if user.blank?

-    TopicEmbed.import_remote(user, @embed_url)
+    TopicEmbed.import_remote(@embed_url, user: user)
   end
 end

@blake

8 curtidas