config: raise Finch pool timeout

A pool timeout shorter than the receive timeout
makes race conditions leading to active connections
being killed more likely and laso just doesn’t make
much sense in general.

See: https://github.com/sneako/finch/pull/292

config/config.exs

@@ -181,7 +181,7 @@
 
 # Configures http settings, upstream proxy etc.
 config :pleroma, :http,
-  pool_timeout: :timer.seconds(5),
+  pool_timeout: :timer.seconds(60),
   receive_timeout: :timer.seconds(15),
   proxy_url: nil,
   user_agent: :default,

lib/pleroma/http/adapter_helper.ex

@@ -6,7 +6,7 @@ defmodule Pleroma.HTTP.AdapterHelper do
   @moduledoc """
   Configure Tesla.Client with default and customized adapter options.
   """
-  @defaults [name: MyFinch, pool_timeout: 5_000, receive_timeout: 5_000]
+  @defaults [name: MyFinch, pool_timeout: 60_000, receive_timeout: 15_000]
 
   @type proxy_type() :: :socks4 | :socks5
   @type host() :: charlist() | :inet.ip_address()

lib/pleroma/http/adapter_helper/default.ex

@@ -10,7 +10,7 @@ defmodule Pleroma.HTTP.AdapterHelper.Default do
   @spec options(keyword(), URI.t()) :: keyword()
   def options(opts, _uri) do
     proxy = Pleroma.Config.get([:http, :proxy_url])
-    pool_timeout = Pleroma.Config.get([:http, :pool_timeout], 5000)
+    pool_timeout = Pleroma.Config.get([:http, :pool_timeout], 60_000)
     receive_timeout = Pleroma.Config.get([:http, :receive_timeout], 15_000)
 
     opts

test/pleroma/http/adapter_helper_test.exs

@@ -50,7 +50,7 @@ test "should not override conn_opts if set" do
   describe "timeout settings" do
     test "should default to 5000/15000" do
       options = AdapterHelper.options(%URI{host: ~c"somewhere"})
-      assert options[:pool_timeout] == 5000
+      assert options[:pool_timeout] == 60_000
       assert options[:receive_timeout] == 15_000
     end