stop using peer.IDFromString

[marten-seemann]

peer.IDFromString was removed from libp2p, as it is almost always used incorrectly, see libp2p/go-libp2p-core#274.

This repo uses it in

go-ipns/record.go

Line 49 in 500f0a8

pid, err := peer.IDFromString(pidString)

I'm not 100% sure if it's used correctly here or not, so I can't submit a fix. This needs to be resolved before we can update the IPFS stack to go-libp2p v0.22.0.

[lidel]

I believe the pidString is binary:

• https:/ipfs/specs/blob/main/IPNS.md#routing-record defines Routing record as /ipns/BINARY_ID
• pidString is the binary part:
  

  go-ipns/record.go

  Line 36 in 500f0a8

  ns, pidString, err := record.SplitKey(key)

So we want peer.IDFromBytes([]byte(pidString)) here. Mind opening a fix @marten-seemann?