[Memo]: drop peer through reference

[rjl493456442]

We have quite a bit of code in the eth handler that operates on peer IDs(e.g. removePeer(id))

Problem

But there's an inherent race in this design. Peer does something stupid, I call drop peer. Before the drop proceeds, peer disconnects and reconnects.
Drop gets sheduled after a reconnect, since it's the same peer id, the peer gets nuked off again or similar weird races.

Solution

I think in general this was a wrong design to address/drop/whatnot peers by id instead of passing around the actual reference to them.

[DavidRomanovizc]

Hi, @rjl493456442, there is no activity here for the last 3 years, let's close this issue for now and open it again if necessary :)