Bug 52353 - Session stays alive after Browser is closed
Summary: Session stays alive after Browser is closed
Status: CLOSED FIXED
Alias: None
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
Version: UCS 4.4
Hardware: Other All
: P5 normal
Target Milestone: UCS 4.4-8-errata
Assignee: Dirk Wiesenthal
QA Contact: Florian Best
URL:
Keywords:
Depends on:
Blocks: 53508
  Show dependency treegraph
 
Reported: 2020-11-12 08:23 CET by Thorsten
Modified: 2021-08-25 17:46 CEST (History)
3 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.057
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Customer ID: 56344
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thorsten univentionstaff 2020-11-12 08:23:31 CET
When closing down the browser the portal session stays alive causing the user being logged in when he restarts the browser (while the session is not expired).

This seems not to be the expected behaviour for the user (unless he explicitly chose an option like "keep me logged in"), even though Ingo told me it works as designed.

For customers we require at least the option to define a system wide setting that changes the sessions behaviour, so the session cookie gets invalidated when the browser is closed. Still there should be a timeout of the session itself, but that would need to be checked by the backend.

In an additional step there could be a third system-wide option offered like "keep me logged in", where the user can select during the login process to keep the session alive, even when the browser is closed. This session would stay alive for a also system wide defined number of days.
Comment 1 Thorsten univentionstaff 2021-03-08 16:41:06 CET
This issue is about the Portal *and* UMC session managed by "UMCSessionId".

The current behaviour stays the default behaviour so the updated doesn't impose a functional change onto existing customer instances.

We would need an UCR variable that allows to set the new behaviour.

I understood that there is actually a session timeout managed in the backend, so we really only need the option to switch the UMCSessionId cookie to be a real session cookie that gets purged when the browser is closed.

See also https://forge.univention.org/bugzilla/show_bug.cgi?id=52873 for the SAML side of things.
Comment 2 Dirk Wiesenthal univentionstaff 2021-06-24 00:24:28 CEST
[4.4-8 0523fc321d] Bug #52353: Login cookie can now be configured to be deleted when browser is closed
 2 files changed, 16 insertions(+), 4 deletions(-)

(+ Changelog + YAML)

[umc/http/session/cookie]
If set, the the login cookie is a session cookie: Closing the browser will delete the cookie, effectively logging out the user.

Also tested with SAML.

Package: univention-management-console
Version: 11.0.6-12A~4.4.0.202106240018
Branch: ucs_4.4-0
Scope: errata4.4-8
Comment 3 Dirk Wiesenthal univentionstaff 2021-06-24 00:36:16 CEST
I will cherry-pick 0523fc321d for 5.0-0 once the bug is verified for 4.4-8.
Comment 4 Florian Best univentionstaff 2021-06-24 09:52:27 CEST
(In reply to Dirk Wiesenthal from comment #3)
> I will cherry-pick 0523fc321d for 5.0-0 once the bug is verified for 4.4-8.

Since UCS 5.0 is release you have to clone the bug.
Comment 6 Dirk Wiesenthal univentionstaff 2021-06-24 12:04:21 CEST
[4.4-8 2757e5c334] Bug #52353: Rename UCRV
 2 files changed, 2 insertions(+), 2 deletions(-)

umc/http/session/cookie -> umc/http/enforce-session-cookie

Package: univention-management-console
Version: 11.0.6-13A~4.4.0.202106241201
Branch: ucs_4.4-0
Scope: errata4.4-8
Comment 7 Florian Best univentionstaff 2021-06-24 12:43:47 CEST
OK: UCR Variable naming
OK: cookies are send as session-cookie without expiry date
OK: UMCLang cookie is still a 5-years cookie