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

Restore data from expired forms #105

Merged
merged 4 commits into from
Jan 1, 2020

Conversation

carlobeltrame
Copy link
Member

@carlobeltrame carlobeltrame commented Dec 24, 2019

When a user leaves e.g. the observation form in Qualix open for a long time (more than 2 hours with the default configuration), he gets logged out automatically because his session expires on the server side. Unfortunately, the form in the browser is still visible by then, and the user can still type in it and try to submit the form, leading to a loss of all entered data when the user is redirected to the login page. This seems to happen regularly in courses, since we got multiple reports about it during a single one-week course. People left the form open overnight or during blocks they taught.

This PR fixes this issue, by saving the submitted form content (in the new session) until the user logs back in. (If the user clicks the submit button on the form and then waits for another 2 hours before logging in, the entered data is permanently lost)

Obsoletes #74. While we still rely on the users to (manually or automatically) lock their screens when they leave the computer, the underlying issue of data loss in #74 is addressed here. Logging out the user after 15 minutes is not a secure solution anyways now that we have the MiData OAuth passwordless login.

@carlobeltrame carlobeltrame merged commit a92da94 into master Jan 1, 2020
@carlobeltrame carlobeltrame deleted the restore-data-from-expired-forms branch January 1, 2020 15:02
@carlobeltrame carlobeltrame mentioned this pull request Jan 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant