Support for /id endpoint

[rs-ipfs]

Support for /id endpoint

[koivunej]

I'll look into this and setting up the node at the same time. At the moment I do not know of a pure-rust RSA keygen, but it then again, for testing purposes at least at the moment, generating an ed25519 key on init --bits 2048 --profile test cannot hurt too much.

[koivunej]

Got stuck yesterday trying to rig up the rsa keygen without using openssl but had to go with that eventually.

[koivunej]

#101 with the simplest /id without ?arg=another_peerid.

[koivunej]

Current status is:

• the testing the ?arg=another_peerid requires #77
• even the local response is using plain multiaddrs without the /ipfs/PeerId suffix

It's probably best to append the /ipfs/PeerId in the responses so that the following multi-node tests using /swarm/connect will be connecting to (PeerId, Multiaddr) or Some(peer_id), multiaddr which way the api will be at that point.

Current plan for implement the identify_other(PeerId) is to cache the identify responses once they come in per PeerId, drop cached values on disconnect and allow awaiting for the response arriving. Unsure if the current libp2p_identify would allow asking the identify on demand, or if that'd even be preferred.

Testing plan currently is to setup three rust-ipfs nodes (a, b, c), connect (a, b), connect (b, c) then ask a to identify_other(c), this requires the mdns disabling at least and kademlia bootstrapping which we do not do, not sure if it's launched automatically on first connection.

[koivunej]

It's probably best to append the /ipfs/PeerId in the responses [...]

This is now drafted in #110 as more up to date /p2p/ suffixes. Also created #105 as the #77 came out without support for peer_ids (#104). Currently discussing how to proceed, as I believe the integration test (quoted below) and support for identify_other will likely yield a solution to #105 as well as earth some things we need to fix in order to make libp2p_kad work for us.

Testing plan currently is to setup three rust-ipfs nodes (a, b, c), connect (a, b), connect (b, c) then ask a to identify_other(c), this requires kademlia bootstrapping which we do not do, not sure if it's launched automatically on first connection.

Removed mdns part as mdns toggling was already in #104.

[koivunej]

A new issue has been split, and the impl in #101, so closing.