Also use http_proxy if valid (#42)

The change introduced in #24 could also be applied to the HTTP_PROXY environment variables.
philss · Dec 10, 2022 · 672a985 · 672a985
1 parent b001ec2
commit 672a985
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lib/rustler_precompiled.ex b/lib/rustler_precompiled.ex
@@ -638,10 +638,11 @@ defmodule RustlerPrecompiled do
     {:ok, _} = Application.ensure_all_started(:inets)
     {:ok, _} = Application.ensure_all_started(:ssl)
 
-    if proxy = System.get_env("HTTP_PROXY") || System.get_env("http_proxy") do
-      Logger.debug("Using HTTP_PROXY: #{proxy}")
-      %{host: host, port: port} = URI.parse(proxy)
+    proxy = System.get_env("HTTP_PROXY") || System.get_env("http_proxy")
 
+    with true <- is_binary(proxy),
+         %{host: host, port: port} when is_binary(host) and is_integer(port) <- URI.parse(proxy) do
+      Logger.debug("Using HTTP_PROXY: #{proxy}")
       :httpc.set_options([{:proxy, {{String.to_charlist(host), port}, []}}])
     end