Bug 54729 – tools.status('loggedIn') not detected correctly anymore

Bug 54729 - tools.status('loggedIn') not detected correctly anymore


Summary:	tools.status('loggedIn') not detected correctly anymore

Status:	NEW

Product:	UCS
Classification:	Unclassified
Component:	UMC (Generic)
Version:	UCS 4.4
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	UMC maintainers
QA Contact:	UMC maintainers

URL:
Keywords:

Depends on:	52321
Blocks:
	Show dependency tree / graph

Reported:	2022-05-06 16:27 CEST by Florian Best
Modified:	2022-05-06 16:28 CEST (History)
CC List:	0 users

See Also:	54728
What kind of report is it?:	Development Internal
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Florian Best

2022-05-06 16:27:54 CEST

Since we changed the get/session-info polling from 30 seconds to the time shortly before the session should terminate, we don't detect anymore if there was an actually session timeout (due to other reasons) and nothing sets `tools.status('loggedIn')` to false anymore.

We should simply add a:
login.onLogin(function() { tools.status('loggedIn', true)});
login.onLogout(function() { tools.status('loggedIn', false)});

somewhere in the main.js.

+++ This bug was initially created as a clone of Bug #52321 +++

Currently we are doing every 30 seconds a get/session-info request which asks for the session validity and checks if one is still logged in. If not one is redirected to the passive iframe SAML login to update the SAML assertion, which is only 5 minutes valid, and is required for LDAP and PAM authentication at the UMC-Server.

We should reduce the polling to time_remaining - 30 = 270.