From 0f667761a9349a852c549c0bfb846b793607e397 Mon Sep 17 00:00:00 2001 From: Mark Felder Date: Fri, 6 Jun 2025 13:00:54 -0700 Subject: [PATCH] The ap_id is a URL, so we can just pass that to set_reachable/1 Also only bother attempting to mark reachable if it was known to be unreachable --- lib/pleroma/workers/remote_fetcher_worker.ex | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/lib/pleroma/workers/remote_fetcher_worker.ex b/lib/pleroma/workers/remote_fetcher_worker.ex index 5f57ec2d7..0cc480c02 100644 --- a/lib/pleroma/workers/remote_fetcher_worker.ex +++ b/lib/pleroma/workers/remote_fetcher_worker.ex @@ -12,13 +12,9 @@ defmodule Pleroma.Workers.RemoteFetcherWorker do def perform(%Job{args: %{"op" => "fetch_remote", "id" => id} = args}) do case Fetcher.fetch_object_from_id(id, depth: args["depth"]) do {:ok, _object} -> - # Mark the server as reachable since we successfully fetched an object - case URI.parse(id) do - %URI{host: host} when not is_nil(host) -> - Instances.set_reachable("https://#{host}") - - _ -> - :ok + unless Instances.reachable?(id) do + # Mark the server as reachable since we successfully fetched an object + Instances.set_reachable(id) end :ok