L'intégration est CASSÉE dans 3.0.4 (stable)

Bug
7

Alors que main a ceci

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

et stable a ceci

TopicEmbed.import_remote(user, @embed_url)

Notez l’ordre des paramètres.

Maintenant, le backport du correctif de sécurité a modifié la signature de la fonction sur stable vers le nouvel ordre des paramètres, donc

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

est devenu

def self.import_remote(url, opts = nil)

et maintenant le paramètre url reçoit un objet User.

Changer l’appel de fonction résout le problème

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 « J'aime »