refactoring
- rename addToTargets to addTarget
- immediately add to cm.targets; the phasing was unnecessary
- don't normalise addresses in addTarget - they are already normalised
get rid of connection attempt count tracking
always notify ConnectionMaker of terminated connections
otherwise CM might have them stuck in the 'attempting' state