fix(http_client): class level execute should default to proxy use

without forcing it on manually initialized classes

Author: stakach

shard.yml

@@ -1,5 +1,5 @@
 name: connect-proxy
-version: 2.0.1
+version: 2.0.2
 license: MIT
 crystal: ">= 0.36.1"
 

src/connect-proxy/http_client.cr

@@ -20,30 +20,17 @@ module ConnectProxy::ProxyHTTP
       yield new(uri, tls, ignore_env)
     end
 
-    # for some reason previous_def doesn't work for these when extending HTTP::Client
-    {% begin %}
-      {% if @type.stringify != "HTTP::Client" %}
-        def self.new(io : IO, host : String = "", port : Int32 = 80)
-          inst = super(io, host, port)
-
-          if ConnectProxy.behind_proxy?
-            inst.set_proxy ConnectProxy.new(*ConnectProxy.parse_proxy_url)
-          end
-
-          inst
-        end
-
-        def self.new(host : String, port : Int32? = nil, tls : TLSContext = nil)
-          inst = super(host, port, tls)
-
-          if ConnectProxy.behind_proxy?
-            inst.set_proxy ConnectProxy.new(*ConnectProxy.parse_proxy_url)
-          end
-
-          inst
+    def self.new(host : String, port = nil, tls : TLSContext = nil, &)
+      client = new(host, port, tls)
+      begin
+        if ConnectProxy.behind_proxy?
+          client.set_proxy ConnectProxy.new(*ConnectProxy.parse_proxy_url)
         end
-      {% end %}
-    {% end %}
+        yield client
+      ensure
+        client.close
+      end
+    end
   end
 
   def set_proxy(proxy : ConnectProxy = nil)