Bug 57466 - OIDC logout not correctly implemented
Summary: OIDC logout not correctly implemented
Status: CLOSED DUPLICATE of bug 57516
Alias: None
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
Version: UCS 5.0
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 5.0-9-errata
Assignee: Christian Castens
QA Contact: Marius Meschter
URL: https://git.knut.univention.de/univen...
Keywords:
Depends on:
Blocks: 57464
  Show dependency treegraph
 
Reported: 2024-07-25 10:21 CEST by Marius Meschter
Modified: 2025-03-21 14:26 CET (History)
4 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Customer ID: 02149
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marius Meschter univentionstaff 2024-07-25 10:21:33 CEST
In the UMC OIDC implementation the logout is not correctly implemented. 
After an RP initiated logout, the post_logout_redirect_uri points to the /oidc/logout-done endpoint which then performs the final logout steps. However this is not what this post_logout_redirect_uri should be used for. After a redirect to the OP by the RP the User will optionally confirm the logout at the OP. 
When it has done this the OP will do a frontchannel/backchannel logout at the RPs. This is the point at which the actual logout takes place at the UMC, either through frontchannel or backchannel logout. The UMC supports both so it's up to the OP to decide. Optionally, according to the spec, the actual logout can also occur before the redirection to the OP.
Freeing up the 'post_logout_redirect_uri' would mean that it can be freed to be configured by the Admin.
Comment 2 Christian Castens univentionstaff 2024-10-15 10:32:28 CEST
This is fixed in the merge request for #57516

*** This bug has been marked as a duplicate of bug 57516 ***