ReverseProxy tesla client: remove handling of old_conn
authorrinpatch <rinpatch@sdf.org>
Sun, 26 Jul 2020 17:44:26 +0000 (20:44 +0300)
committerrinpatch <rinpatch@sdf.org>
Sun, 26 Jul 2020 17:52:00 +0000 (20:52 +0300)
This is no longer relevant because we use a custom
FollowRedirects middleware now

lib/pleroma/reverse_proxy/client/tesla.ex

index 84addc404e485db7193d191ff07f0869b099e44f..d5a339681a752d6ae301ed08c33a49ca4b36353f 100644 (file)
@@ -45,15 +45,8 @@ defmodule Pleroma.ReverseProxy.Client.Tesla do
   @impl true
   @spec stream_body(map()) ::
           {:ok, binary(), map()} | {:error, atom() | String.t()} | :done | no_return()
-  def stream_body(%{pid: pid, opts: opts, fin: true}) do
-    # if connection was reused, but in tesla were redirects,
-    # tesla returns new opened connection, which must be closed manually
-    if opts[:old_conn], do: Tesla.Adapter.Gun.close(pid)
-    # if there were redirects we need to checkout old conn
-    conn = opts[:old_conn] || opts[:conn]
-
-    if conn, do: :ok = ConnectionPool.release_conn(conn)
-
+  def stream_body(%{pid: pid, fin: true}) do
+    ConnectionPool.release_conn(pid)
     :done
   end