Enabling ShareTCPListener triggers excessive logging and high CPU usage

[acha-bill]

ShareTCPListener was introduced in PR #5069 to allow Bee to share the same listen address between TCP and WebSocket transports. This is required to enable WebSocket support.

After enabling ShareTCPListener, we observed that nodes started logging sleeping to refresh overdraft balance excessively. This drove CPU very high.

The image below compares the log volume of a node with and without ShareTCPListener enabled (ignore warmup time)

The task is to investigate the behavior triggered by enabling ShareTCPListener.

Note that ShareTCPListener is currently opt-in in libp2p, but it is planned to become the default behavior in future libp2p versions.

[martinconic]

As I investigated this issue, it could be that upgrading github.com/libp2p/go-libp2p to v0.42.0 , might have fixed the issues we had with github.com/libp2p/go-libp2p v0.33.2

[nugaon]

maybe is it connected to p2p headers? because only swap protocol uses that which can cause this issue.

[martinconic]

We removed for now ShareTCPListener.