p2p: fix logging peer address

[ebuchman]

Pointed out in #2721 (comment)

module=p2p peer=0xb78f00

It's because we're passing a function pointer rather than calling the function in:

tendermint/p2p/switch.go

Line 619 in 6e9aee5

p.SetLogger(sw.Logger.With("peer", p.NodeInfo().NetAddress().String))

We should just pass the NetAddress() there, no need to call String().

But note even if we do that, we still lose the ID. That's because in

tendermint/p2p/netaddress.go

Lines 316 to 321 in 6e9aee5

	func removeProtocolIfDefined(addr string) string {
	if strings.Contains(addr, "://") {
	return strings.Split(addr, "://")[1]
	}
	return addr

we expect the protocol to be at the beginning, but it could come after the @, eg. 76e35576b8dd1cf8049fea382fda8ac621a9c6ee@tcp://0.0.0.0:26656. So we need fix this too

[ebuchman]

Fixed in #2797