Why GRPC.Adapter.Gun.connect/2 returns {:error, String.t()} on error?

[bszaf]

I would like to handle some connect errors, that are returned from :gun in specific way.

Is there any reason why the error is converted to string?

grpc/lib/grpc/adapter/gun.ex

Lines 68 to 71 in d8a58cc

	{:error, reason} ->
	:gun.shutdown(conn_pid)
	{:error, "Error when opening connection: #{inspect(reason)}"}
	end

Would it be okay to return just {:error, reason} instead of creating String error message?

Otherwise, for compatibility reasons, can another option be added to GRPC.Adapter.Gun.connect/2, that will prevent from converting error into String?

[tony612]

String is to keep consistent with {:error, "Error when opening connection: protocol #{proto} is not http2"}. Let me change this.