Remove caching because it does not affect performance and may be even worse in some...

diff --git a/lib/pleroma/web/metadata.ex b/lib/pleroma/web/metadata.ex
index ddc74fb0d191e3df23ec24b9f58be1c348d6f82c..d859dfd8b938705e63a4aaaf9a9ee0a4e8eea278 100644 (file)
--- a/lib/pleroma/web/metadata.ex
+++ b/lib/pleroma/web/metadata.ex
@@ -2,31 +2,6 @@ defmodule Pleroma.Web.Metadata do
   alias Phoenix.HTML
 
   @providers Pleroma.Config.get([__MODULE__, :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
-    key = :erlang.term_to_binary(user)
-
-    Cachex.fetch!(:metadata_cache, key, fn _key ->
-      {:commit, build_tags(params)}
-    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(@providers, "", fn parser, acc ->

diff --git a/lib/pleroma/web/router.ex b/lib/pleroma/web/router.ex
index 1ecd4aee1a03a8eae334d0c7254eb335c9c1cc76..25e866c4897746b96a66ed990632e2613c026368 100644 (file)
--- a/lib/pleroma/web/router.ex
+++ b/lib/pleroma/web/router.ex
@@ -534,7 +534,7 @@ defmodule Fallback.RedirectController do
 
   def redirector_with_meta(conn, params) do
     {:ok, index_content} = File.read(index_file_path())
-    tags = Metadata.get_cached_tags(params)
+    tags = Metadata.build_tags(params)
     response = String.replace(index_content, "<!--server-generated-meta-->", tags)
 
     conn