We're planning on running the webapp, keyserver, and landing page in isolation in separate containers for our prod self-host setup.
This means we should remove this line server.get('/invite/:secret', inviteResponder);
Differential D12808
[keyserver] remove unnecessary inviteResponder line in keyserver.js will on Jul 19 2024, 2:15 PM. Authored by Tags None Referenced Files
Subscribers
Details We're planning on running the webapp, keyserver, and landing page in isolation in separate containers for our prod self-host setup. This means we should remove this line server.get('/invite/:secret', inviteResponder); flow check. I believe we plan on testing this on prod and check if the invite links still works. We can fully remove if confirmed working
Diff Detail
Event TimelineComment Actions Looks like some context in D8255... web doesn't really need this route, landing definitely does, and it's not clear if it's needed at the root either. I think it's fine to remove given we'll be running landing on the apex domain |