refactor: make debug-print of StreamProtocol more concise

[thomaseizinger]

Description

The fmt::Debug implementation of a type should in most cases reveal its internal structure. StreamProtocol is likely to be debug-printed a lot and in many cases, the only contract is the fmt::Debug impl. The internals of StreamProtocol only exist for performance reasons to avoid allocations for statically-known protocol strings. Revealing this implementation detail isn't particularly beneficial to end users. At the same time, the current implementation is very noise.

Previously, the protocols field of an identify::Info would e.g. read as:

protocols: [StreamProtocol { inner: Right("/ipfs/id/1.0.0") }, StreamProtocol { inner: Right("/ipfs/id/push/1.0.0") }, StreamProtocol { inner: Right("/ipfs/kad/1.0.0") }]

With this patch, it reads as:

protocols: ["/ipfs/id/1.0.0", "/ipfs/kad/1.0.0", "/ipfs/id/push/1.0.0"]

Notes & open questions

Change checklist

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• A changelog entry has been made in the appropriate crates

[mxinden]

🚀