Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ability to specify JWT secret used by server #765

Open
wsanchez opened this issue Nov 8, 2022 · 2 comments
Open

Add ability to specify JWT secret used by server #765

wsanchez opened this issue Nov 8, 2022 · 2 comments

Comments

@wsanchez
Copy link
Member

wsanchez commented Nov 8, 2022

Add ability to specify JWT secret used by server, so that we can share the secret across application instances (required for >1 instance) and the Clubhouse (required for shared sign-in with Clubhouse).

@wsanchez wsanchez added this to the 2023 Burning Man Event milestone Nov 8, 2022
@wsanchez
Copy link
Member Author

Config is in place but restarting the server should not require logging in again… but it still does.

@srabraham
Copy link
Member

Config is in place but restarting the server should not require logging in again… but it still does.

Looks like I wound up here yesterday! #1363 (comment)

The legacy IMS app relies entirely on the TWISTED_SESSION cookie to authenticate the user. If the server restarts, that TWISTED_SESSION is no longer valid. It's conceivable that the legacy app could instead also get itself a bearer token on login, then use that to create a new Twisted session anytime the old session is no longer valid. I've poked around with that a bit.

The new IMS app uses bearer tokens instead, but it also sends over the TWISTED_SESSION cookie to the server. If that cookie value is important even in the new app, then this change might be necessary on the server-side. #1363 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants