This creates a new thread pool for each login request. That's a terrible idea. If anything create it once statically and re-use it. Or just perform your actions in the calling thread as you're just issuing a .get() on the future anyway. Again, it's needless complexity.
Why fork threads only to immediately join on them? This is just sync code with extra context switching and fatter stack traces.
> https://github.com/VOTEism/server/blob/d8d3752af99d7f4c25c0b...
This creates a new thread pool for each login request. That's a terrible idea. If anything create it once statically and re-use it. Or just perform your actions in the calling thread as you're just issuing a .get() on the future anyway. Again, it's needless complexity.