My diffs (D11012 and D11013) don't handle the "Handlers"... arguably those should get the keyserverID from the Socket too

We could land this diff as-is for those files, but arguably it would be easy (and better) to have Socket pass in keyserverID and to use that here

Actually, that will be replaced by a call to fetchIdentityInfo in ENG-6633
Also I think that would be logically incorrect, because a keyserver wants to return their values, so they should put them under their own id, not the authoritative keyservers id

This is a little bit fragile - we don't enforce this relationship. Maybe we can add an invariant

invariant(!process.env['KEYSERVER'], '...')

Or maybe we can check this condition instead of just hasConfig

if (process.env['KEYSERVER'] || !hasConfig())

Does keyserver use this when it does server-side rendering of web?

Hmm I didn't experience the above invariant trowing, so I don't think so. And I was able to login to the app and send messages using this code.
Based on D5983 it looks like we are only rendering the loading screen.

This comment made sense when we were calling this function top level, it doesn't anymore.
This file is being read on the keyserver (if I add a top level console.log in this file, it shows up in the keyserver console), I am not sure why, but the function should not be called