search

koinos / koinos-p2p

The p2p microservice orchestrates the distribution of blocks and transactions between peers.
MIT License
6 stars 4 forks source link

[BUG]: Peer ID logging display #181

Closed sgerbino closed 2 years ago

sgerbino commented 2 years ago

Is there an existing issue for this?

Current behavior

The connected peer ID is not separated from p2p with a /.

2021-11-02 18:03:18.203977 (p2p.Koinos) [node/node.go:260] <info>: Connected peers:
2021-11-02 18:03:18.204009 (p2p.Koinos) [node/node.go:262] <info>:  - /ip4/x.x.x.x/tcp/58388/p2pQmTZnodiJRbmrkbCexxosb9QPb7UFfgV51hJXacpgSrfC2

Expected behavior

The peer ID should be displayed like this: /ip4/x.x.x.x/tcp/58388/p2p/QmTZnodiJRbmrkbCexxosb9QPb7UFfgV51hJXacpgSrfC2

Steps to reproduce

Run a Koinos cluster and look at the p2p logs for connected peers.

Environment

- OS: All

Anything else?

Add the forward slash in this statement.

https://github.com/koinos/koinos-p2p/blob/9cac5509f1e6a8be534dde789b81e740a783ad50/internal/node/node.go#L262