Fedora Account System
Red Hat Associate
Red Hat Customer
Created attachment 1603841 [details] Screenshot showing the error when a normal logout has been performed. Description of problem: intermittently getting "Session was timed out due to inactivity. Please log in again." error on the login screen when logging out and also when logging out and logging in quickly Version-Release number of selected component (if applicable): 5.11.0.18.20190806180636_1dd6378 How reproducible: frequently Steps to Reproduce: 1. Login to the OPS UI 2. Navigate to a few different pages 3. Select Username > Logout Actual results: The error about session timeout is displayed Expected results: No timeout error is displayed Additional info:
Please assess the impact of this issue and update the severity accordingly. Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition. If it's something like a tracker bug where it doesn't matter, please set the severity to Low.
https://github.com/ManageIQ/manageiq-ui-classic/pull/6065
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/32b4b1593752025d563d425617c16fb60865c813 commit 32b4b1593752025d563d425617c16fb60865c813 Author: Martin Hradil <mhradil> AuthorDate: Tue Aug 20 07:10:20 2019 -0400 Commit: Martin Hradil <mhradil> CommitDate: Tue Aug 20 07:10:20 2019 -0400 Prevent login redirect with ?timeout during logout When logging out via the Logout button, when there are multiple HTTP requests happening, the logout can cause API requests to fail with a 401 before the browser redirects to the login screen, causing the response handler to call redirectLogin, to redirec to the login screen with a "session timed out" message. We should not be claiming a timeout *during* logout, adding a check for that. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1741283 app/assets/javascripts/miq_application.js | 4 + app/assets/javascripts/miq_global.js | 1 + app/views/layouts/_user_options.html.haml | 2 +- 3 files changed, 6 insertions(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq-ui-classic/ivanchuk: https://github.com/ManageIQ/manageiq-ui-classic/commit/8b17db500718e2c81ce760c5567b6e3e709ace77 commit 8b17db500718e2c81ce760c5567b6e3e709ace77 Author: Milan Zázrivec <mzazrivec> AuthorDate: Wed Aug 21 06:08:01 2019 -0400 Commit: Milan Zázrivec <mzazrivec> CommitDate: Wed Aug 21 06:08:01 2019 -0400 Merge pull request #6065 from himdel/bz1741283-logout Prevent login redirect with ?timeout during logout (cherry picked from commit 86fafd124d25addda12d715b5d7d8ec9f53614e5) https://bugzilla.redhat.com/show_bug.cgi?id=1741283 app/assets/javascripts/miq_application.js | 4 + app/assets/javascripts/miq_global.js | 1 + app/views/layouts/_user_options.html.haml | 2 +- 3 files changed, 6 insertions(+), 1 deletion(-)
Hi, I am not able to reproduce it on 5.11.0.18, can you help me reproduce it first?
Parthvi, the most likely places this can happen is places which do a http request for data, and wait for the response before issuing a follow-up request. On those, "all" you have to do is click the Logout button before the second wave of requests gets triggered. (Or even after, but then you'd rely on server reordering the requests.) You *may* have luck using the browser's inspector to emulate a slow network. It may also help to have many entities in the DB. (A form loading a list of cloud tenants will be slower when there's thousands of those, making it easier to trigger the issue.) (I was not able to reproduce this without code changes, so if there's a way, Brant will have to share it.)
Hi Brant, can you help me reproduce this BZ? Thanks for the help Martin, I'll see if I can reproduce it.
FIXED. Verified on 5.11.0.22. To reproduce this BZ, I added multiple providers in the appliance and logged in, and before all the widgets loaded on the dashboard(before the spinner disappeared) logged out of the appliance. I was able to reproduce this on 5.11.0.18 and verify it in 5.11.0.22.