adapter renaming to adapter_helper

This commit is contained in:
Alexander Strizhakov 2020-03-03 18:53:44 +03:00
parent 8854770fc4
commit f98ee730f0
No known key found for this signature in database
GPG Key ID: 022896A53AEF1381
8 changed files with 44 additions and 43 deletions

View File

@ -2,7 +2,7 @@
# Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/> # Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/>
# SPDX-License-Identifier: AGPL-3.0-only # SPDX-License-Identifier: AGPL-3.0-only
defmodule Pleroma.HTTP.Adapter do defmodule Pleroma.HTTP.AdapterHelper do
alias Pleroma.HTTP.Connection alias Pleroma.HTTP.Connection
@type proxy :: @type proxy ::

View File

@ -2,10 +2,10 @@
# Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/> # Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/>
# SPDX-License-Identifier: AGPL-3.0-only # SPDX-License-Identifier: AGPL-3.0-only
defmodule Pleroma.HTTP.Adapter.Gun do defmodule Pleroma.HTTP.AdapterHelper.Gun do
@behaviour Pleroma.HTTP.Adapter @behaviour Pleroma.HTTP.AdapterHelper
alias Pleroma.HTTP.Adapter alias Pleroma.HTTP.AdapterHelper
require Logger require Logger
@ -28,7 +28,7 @@ def options(connection_opts \\ [], %URI{} = uri) do
|> Keyword.merge(Pleroma.Config.get([:http, :adapter], [])) |> Keyword.merge(Pleroma.Config.get([:http, :adapter], []))
|> add_original(uri) |> add_original(uri)
|> add_scheme_opts(uri) |> add_scheme_opts(uri)
|> Adapter.maybe_add_proxy(Adapter.format_proxy(proxy)) |> AdapterHelper.maybe_add_proxy(AdapterHelper.format_proxy(proxy))
|> maybe_get_conn(uri, connection_opts) |> maybe_get_conn(uri, connection_opts)
end end

View File

@ -1,5 +1,5 @@
defmodule Pleroma.HTTP.Adapter.Hackney do defmodule Pleroma.HTTP.AdapterHelper.Hackney do
@behaviour Pleroma.HTTP.Adapter @behaviour Pleroma.HTTP.AdapterHelper
@defaults [ @defaults [
connect_timeout: 10_000, connect_timeout: 10_000,
@ -17,7 +17,7 @@ def options(connection_opts \\ [], %URI{} = uri) do
|> Keyword.merge(Pleroma.Config.get([:http, :adapter], [])) |> Keyword.merge(Pleroma.Config.get([:http, :adapter], []))
|> Keyword.merge(connection_opts) |> Keyword.merge(connection_opts)
|> add_scheme_opts(uri) |> add_scheme_opts(uri)
|> Pleroma.HTTP.Adapter.maybe_add_proxy(proxy) |> Pleroma.HTTP.AdapterHelper.maybe_add_proxy(proxy)
end end
defp add_scheme_opts(opts, %URI{scheme: "http"}), do: opts defp add_scheme_opts(opts, %URI{scheme: "http"}), do: opts

View File

@ -18,7 +18,7 @@ defmodule Pleroma.HTTP.Connection do
require Logger require Logger
alias Pleroma.Config alias Pleroma.Config
alias Pleroma.HTTP.Adapter alias Pleroma.HTTP.AdapterHelper
@doc """ @doc """
Merge default connection & adapter options with received ones. Merge default connection & adapter options with received ones.
@ -50,9 +50,9 @@ def after_request(opts), do: adapter().after_request(opts)
defp adapter do defp adapter do
case Application.get_env(:tesla, :adapter) do case Application.get_env(:tesla, :adapter) do
Tesla.Adapter.Gun -> Adapter.Gun Tesla.Adapter.Gun -> AdapterHelper.Gun
Tesla.Adapter.Hackney -> Adapter.Hackney Tesla.Adapter.Hackney -> AdapterHelper.Hackney
_ -> Adapter _ -> AdapterHelper
end end
end end

View File

@ -2,13 +2,13 @@
# Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/> # Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/>
# SPDX-License-Identifier: AGPL-3.0-only # SPDX-License-Identifier: AGPL-3.0-only
defmodule Pleroma.HTTP.Adapter.GunTest do defmodule Pleroma.HTTP.AdapterHelper.GunTest do
use ExUnit.Case, async: true use ExUnit.Case, async: true
use Pleroma.Tests.Helpers use Pleroma.Tests.Helpers
import ExUnit.CaptureLog import ExUnit.CaptureLog
alias Pleroma.Config alias Pleroma.Config
alias Pleroma.Gun.Conn alias Pleroma.Gun.Conn
alias Pleroma.HTTP.Adapter.Gun alias Pleroma.HTTP.AdapterHelper.Gun
alias Pleroma.Pool.Connections alias Pleroma.Pool.Connections
setup_all do setup_all do

View File

@ -2,12 +2,12 @@
# Copyright © 2017-2019 Pleroma Authors <https://pleroma.social/> # Copyright © 2017-2019 Pleroma Authors <https://pleroma.social/>
# SPDX-License-Identifier: AGPL-3.0-only # SPDX-License-Identifier: AGPL-3.0-only
defmodule Pleroma.HTTP.Adapter.HackneyTest do defmodule Pleroma.HTTP.AdapterHelper.HackneyTest do
use ExUnit.Case use ExUnit.Case
use Pleroma.Tests.Helpers use Pleroma.Tests.Helpers
alias Pleroma.Config alias Pleroma.Config
alias Pleroma.HTTP.Adapter.Hackney alias Pleroma.HTTP.AdapterHelper.Hackney
setup_all do setup_all do
uri = URI.parse("http://domain.com") uri = URI.parse("http://domain.com")

View File

@ -0,0 +1,28 @@
# Pleroma: A lightweight social networking server
# Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/>
# SPDX-License-Identifier: AGPL-3.0-only
defmodule Pleroma.HTTP.AdapterHelperTest do
use ExUnit.Case, async: true
alias Pleroma.HTTP.AdapterHelper
describe "format_proxy/1" do
test "with nil" do
assert AdapterHelper.format_proxy(nil) == nil
end
test "with string" do
assert AdapterHelper.format_proxy("127.0.0.1:8123") == {{127, 0, 0, 1}, 8123}
end
test "localhost with port" do
assert AdapterHelper.format_proxy("localhost:8123") == {'localhost', 8123}
end
test "tuple" do
assert AdapterHelper.format_proxy({:socks4, :localhost, 9050}) ==
{:socks4, 'localhost', 9050}
end
end
end

View File

@ -1,27 +0,0 @@
# Pleroma: A lightweight social networking server
# Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/>
# SPDX-License-Identifier: AGPL-3.0-only
defmodule Pleroma.HTTP.AdapterTest do
use ExUnit.Case, async: true
alias Pleroma.HTTP.Adapter
describe "format_proxy/1" do
test "with nil" do
assert Adapter.format_proxy(nil) == nil
end
test "with string" do
assert Adapter.format_proxy("127.0.0.1:8123") == {{127, 0, 0, 1}, 8123}
end
test "localhost with port" do
assert Adapter.format_proxy("localhost:8123") == {'localhost', 8123}
end
test "tuple" do
assert Adapter.format_proxy({:socks4, :localhost, 9050}) == {:socks4, 'localhost', 9050}
end
end
end