vyzo
vyzo
I think we are overthinking this; if the remote closes the stream we currently do nothing. If we want to allow a retry, then we can allow up to two...
Possibly not having observed public IPv6 addrs to exchange, which breaks everything -- ref also the other discussion we are having in go-libp2p.
this should be done through the resource mamager, dont hardcode values please.
cc @Stebalien
Some complaints from go-checks aboug go.mod not being tidy ... but that is! Seems it is the examples however.
So the `TestMessageSeenCacheTTLTest` test fails because the behaviour of the seen cache has changed -- it doesn't eagerly aggregate any more and it cleans up with an background goroutine that...
~Hrm, I don't like disabling tests either, so let's try with 1min waits (and see whether we should fix that interval).~ Edit: I don't have the patience for this, disabling...
Thinking more about this test situation -- i think this test simply doesn't belong. It violates abstraction by programming against a specific implementation of the cache which is now gone....
I am not sure we need a behaviour that is only available while the command is active - do we really care all that much about seeing background resolutions displayed...
There is also the implicit relationship to ipns pubsub. We can have the follow command work through pubsub or it can be something that does background resolutions. The former is...