Activity.Search: fallback on status resolution on DB Timeout
authorHaelwenn (lanodan) Monnier <contact@hacktivis.me>
Thu, 12 Aug 2021 08:29:53 +0000 (10:29 +0200)
committerHaelwenn (lanodan) Monnier <contact@hacktivis.me>
Thu, 12 Aug 2021 08:35:32 +0000 (10:35 +0200)
CHANGELOG.md
lib/pleroma/activity/search.ex

index 45a365505756860a3c5a3b272efdd5a7138b0f67..421b6e04312cdf3bb19362b628f307663a345b40 100644 (file)
@@ -14,6 +14,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ### Removed
 
+## Unreleased-patch
+- Mastodon API: Activity Search fallbacks on status fetching after a DB Timeout/Error
+
 ## 2.4.0 - 2021-08-xx
 
 ### Changed
index ed898ba4ff847a79c93c15f6d9b6c7355057d50b..a5923519c3d7434534d848534093cd38144c99f1 100644 (file)
@@ -26,19 +26,23 @@ defmodule Pleroma.Activity.Search do
         :plain
       end
 
-    Activity
-    |> Activity.with_preloaded_object()
-    |> Activity.restrict_deactivated_users()
-    |> restrict_public()
-    |> query_with(index_type, search_query, search_function)
-    |> maybe_restrict_local(user)
-    |> maybe_restrict_author(author)
-    |> maybe_restrict_blocked(user)
-    |> Pagination.fetch_paginated(
-      %{"offset" => offset, "limit" => limit, "skip_order" => index_type == :rum},
-      :offset
-    )
-    |> maybe_fetch(user, search_query)
+    try do
+      Activity
+      |> Activity.with_preloaded_object()
+      |> Activity.restrict_deactivated_users()
+      |> restrict_public()
+      |> query_with(index_type, search_query, search_function)
+      |> maybe_restrict_local(user)
+      |> maybe_restrict_author(author)
+      |> maybe_restrict_blocked(user)
+      |> Pagination.fetch_paginated(
+        %{"offset" => offset, "limit" => limit, "skip_order" => index_type == :rum},
+        :offset
+      )
+      |> maybe_fetch(user, search_query)
+    rescue
+      _ -> maybe_fetch([], user, search_query)
+    end
   end
 
   def maybe_restrict_author(query, %User{} = author) do