From 6af6a9704d2d0a04a43967805368eed419b9dc67 Mon Sep 17 00:00:00 2001
From: "Haelwenn (lanodan) Monnier" <contact@hacktivis.me>
Date: Fri, 23 Feb 2024 01:57:43 +0100
Subject: RemoteFetcherWorker: Make sure {:error, _} is returned on failure

Otherwise jobs are considered a success.
---
 changelog.d/remote-fetcher-error.skip        | 0
 lib/pleroma/workers/remote_fetcher_worker.ex | 7 +++++--
 2 files changed, 5 insertions(+), 2 deletions(-)
 create mode 100644 changelog.d/remote-fetcher-error.skip

diff --git a/changelog.d/remote-fetcher-error.skip b/changelog.d/remote-fetcher-error.skip
new file mode 100644
index 000000000..e69de29bb
diff --git a/lib/pleroma/workers/remote_fetcher_worker.ex b/lib/pleroma/workers/remote_fetcher_worker.ex
index d526a99cb..c26418483 100644
--- a/lib/pleroma/workers/remote_fetcher_worker.ex
+++ b/lib/pleroma/workers/remote_fetcher_worker.ex
@@ -22,8 +22,11 @@ def perform(%Job{args: %{"op" => "fetch_remote", "id" => id} = args}) do
       {:error, :allowed_depth} ->
         {:discard, :allowed_depth}
 
-      _ ->
-        :error
+      {:error, _} = e ->
+        e
+
+      e ->
+        {:error, e}
     end
   end
 
-- 
cgit v1.2.3