Some libp2p streams needs to use transient connections

[aarshkshah1992]

Once libp2p/go-libp2p-transport-upgrader#71 and the Limited Relay work by @vyzo lands, streams need to explicitly tell the Swarm(via a context) that they are okay with using Transient connections.

Streams like Identify and the Hole punching streams should use Transient connections as the Hole punching attempt depends on Identify completion.

[aarshkshah1992]

cc @vyzo

Now that I think about it, all streams except Identify& Hole-Punch should use non-transient connections. I don't think we are missing much by not talking to DHT peers that are behinds Relays anyways. Wdyt ?

[marten-seemann]

We now have transient connections and we're using them in some application protocols.

[wanjunlengfeng]

relayv2.there is a word:check for a limit provided by the relay; if the limit is not nil, then this is a limited.relay connection and we mark the connection as transient.
so I set the value into stat.Transient = false.
"transient connection to peer" dispear .
how do I get the correct set way?

[vyzo]

use the network.UseTransient context option.

[vyzo]

but are you sure this is what you want to do?

[wanjunlengfeng]

when i use relayv2.New(host),Throw an exception “transient connection”.
when i use relayv2.New(host, relayv2.WithLimit(nil)) (set limit=0), all is well.
reservation.LimitData and reservation.LimitDuration =0.
My purpose is to limit the speed, what do I need to do?

[marten-seemann]

@wanjunlengfeng

1. Please don't hijack issues that have been closed for more than a year.
2. This is not a bug report nor a feature request. Please refer to the forums: https://discuss.libp2p.io/.