Fixed race where pending connections were being added without holding mutex - this allowed the same pending connections to be cleared concurrently in another thread which led to segfaults and hangs.
Merge remote-tracking branch 'remotes/main/next' into next
Lock was deadlocking, so replaced with assert for lock to see what is calling this unlocked.
Took out the lock. I think the race is actually being caused by NodeId(NodeId&&) that lack of noexcept yet again ...