connect with http2 protocol may retry to connect

[thenrio]

On req-0.5.0,

The following is ok

iex(1)> Req.get("https://eno.nt", connect_options: [protocols: [:http1]])
{:error, %Req.TransportError{reason: :nxdomain}}

When I use http2 as protocol, then I see retry attempts:

iex(2)> Req.get("https://eno.nt", connect_options: [protocols: [:http2]])

13:57:26.725 [warning] Failed to connect to https://eno.nt:443: non-existing domain
{:error, %Req.HTTPError{protocol: :http2, reason: :disconnected}}
iex(3)>
13:57:27.077 [warning] Failed to connect to https://eno.nt:443: non-existing domain

13:57:27.622 [warning] Failed to connect to https://eno.nt:443: non-existing domain

13:57:28.321 [warning] Failed to connect to https://eno.nt:443: non-existing domain

I think we should not reconnect in this case.

[wojtekmach]

Thanks for the report. These retries are coming from Finch:

iex(1)> Req.get("https://eno.nt", connect_options: [protocols: [:http2]], retry: false)

20:48:14.421 [warning] Failed to connect to https://eno.nt:443: non-existing domain
{:error, %Req.HTTPError{protocol: :http2, reason: :disconnected}}

20:48:14.428 [warning] Failed to connect to https://eno.nt:443: non-existing domain

20:48:15.177 [warning] Failed to connect to https://eno.nt:443: non-existing domain

20:48:15.445 [warning] Failed to connect to https://eno.nt:443: non-existing domain

20:48:17.268 [warning] Failed to connect to https://eno.nt:443: non-existing domain

Could you re-open this in Finch repo? I'll of course be happy to make any required changes in Req. Thanks!