Add a fallback function to handle generic params, just in case

[akkoma] / lib / pleroma / web / metadata.ex

diff --git a/lib/pleroma/web/metadata.ex b/lib/pleroma/web/metadata.ex
index 8903671527067c59de3b31c849940a70e4b97740..23f152e068ca5cffd29e83f4a7c5769a0060203e 100644 (file)
--- a/lib/pleroma/web/metadata.ex
+++ b/lib/pleroma/web/metadata.ex
@@ -2,10 +2,17 @@ defmodule Pleroma.Web.Metadata do
   alias Phoenix.HTML
 
   @parsers Pleroma.Config.get([:metadata, :providers], [])
+  def get_cached_tags(%{activity: activity, user: user} = params) do
+    # We don't need to use the both activity and a user since the object can't change it's content
+    key = "#{:erlang.term_to_binary(user)}#{activity.data["id"]}"
+
+    Cachex.fetch!(:metadata_cache, key, fn _key ->
+      {:commit, build_tags(params)}
+    end)
+  end
 
   def get_cached_tags(%{user: user} = params) do
     # I am unsure how well ETS works with big keys
-    # We don't need to use the both activity and a user since the object can't change it's content
     key = :erlang.term_to_binary(user)
 
     Cachex.fetch!(:metadata_cache, key, fn _key ->
@@ -13,6 +20,14 @@ defmodule Pleroma.Web.Metadata do
     end)
   end
 
+  def get_cached_tags(params) do
+    key = :erlang.term_to_binary(params)
+
+    Cachex.fetch!(:metadata_cache, key, fn _key ->
+      {:commit, build_tags(params)}
+    end)
+  end
+
   def build_tags(params) do
     Enum.reduce(@parsers, "", fn parser, acc ->
       rendered_html =