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

timeouts on authentication should be detected #138

Open
kmccurley opened this issue Jun 7, 2018 · 2 comments
Open

timeouts on authentication should be detected #138

kmccurley opened this issue Jun 7, 2018 · 2 comments

Comments

@kmccurley
Copy link
Member

if a user sits on the "existing"/"new" choice for too long, then authentication will timeout. We need to detect this because at the moment it just shows an empty list.

@kmccurley
Copy link
Member Author

I attempted to fix this on the server side by extending the timeout for PHP session cookies. There are two ways to do this:

  1. change the global setting. This would also extend the timeout on the conference registration system (which I believe is the only part of the site that uses PHP sessions).
  2. change the path for the session cookies on this app and extend the time in the code. This has the unfortunate side effect that it immediately logs everyone out from the app. In order to use this, we need to first push logic that handles authentication timeouts gracefully and shows the modal for logging in again.

@kmccurley
Copy link
Member Author

We've addressed this partially by introducing auto-save. It's also a duplicate of #205

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

No branches or pull requests

1 participant