RemoteFetcherWorker: Make sure {:error, _} is returned on failure

Otherwise jobs are considered a success.
2024-02-23 01:57:43 +01:00 · 2024-02-23 01:57:43 +01:00 · 6af6a9704d
parent f0468697cd
commit 6af6a9704d
2 changed files with 5 additions and 2 deletions
--- a/changelog.d/remote-fetcher-error.skip
+++ b/changelog.d/remote-fetcher-error.skip
--- 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