@xiangli-cmu This seems dangerous. The returned `peers` map will be outside the server lock after it's returned. What did you want to use this for? We'd either have to copy the whole state of the peers or maybe just copy out the names (e.g. `s.PeerNames()`).