Skip to content
This repository has been archived by the owner on Sep 19, 2021. It is now read-only.

Session timeout terminates session early #895

Closed
ryanhofdotgov opened this issue Sep 27, 2018 · 0 comments
Closed

Session timeout terminates session early #895

ryanhofdotgov opened this issue Sep 27, 2018 · 0 comments
Assignees

Comments

@ryanhofdotgov
Copy link
Contributor

What did you do?

Login, accept terms, don't on anything except maybe: navigation menu expand/close, instructions. Don't click on Next or click onto a sub-section page. Wait 1-2 minutes.

What did you expect to see?

No logout until the 15 minute session timeout.

What did you see instead?

Logout.

Additional Information

tick() in TimeoutWarning.jsx does a check against props.lastRefresh the first time it is called, which if you don't change routes by navigation to a page, will be null (evaluated to 0), due to defaultProps of TimeoutWarning – set on initial page load. This incorrectly makes the diff >= to timeout.

To correct, I believe lastRefresh needs to be set to the current time after the first successful login.

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

No branches or pull requests

2 participants