Bug 55507 - [5.0] SAML IdP: Group member comparison should be case insensitive
[5.0] SAML IdP: Group member comparison should be case insensitive
Status: CLOSED INVALID
Product: UCS
Classification: Unclassified
Component: SAML
UCS 5.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: Erik Damrose
Florian Best
:
Depends on: 53723
Blocks: 53432
  Show dependency treegraph
 
Reported: 2022-12-19 13:25 CET by Simon Streit
Modified: 2023-01-09 15:12 CET (History)
10 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?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 3: A User would likely not purchase the product
User Pain: 0.257
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support: Yes
Flags outvoted (downgraded) after PO Review:
Ticket number: 2022120621000191
Bug group (optional): Regression
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Simon Streit univentionstaff 2022-12-27 10:43:29 CET
A customer reported a possible regression in regards to case insensitivity.  Apparantly SAML binddn fails while spotting differences when cases have changed:

SAML binddn does not match: 'uid=john,cn=staff,cn=users,ou=ts,dc=example,dc=com' != 'uid=john,cn=staff,cn=users,ou=TS,dc=example,dc=com'

UCS: 5.0-2 errata471 (UCS@school)
Comment 2 Florian Best univentionstaff 2023-01-02 10:52:26 CET
(In reply to Simon Streit from comment #1)
> SAML binddn does not match:
> 'uid=john,cn=staff,cn=users,ou=ts,dc=example,dc=com' !=
> 'uid=john,cn=staff,cn=users,ou=TS,dc=example,dc=com'
This is just a warning in the log files of the UMC-server which can be ignored. It will be fixed via Bug #44883.

Is there a real problem which can be observed in broken behavior?
Comment 3 Dirk Schnick 2023-01-02 11:19:07 CET
Hi Florian, the endcustomer reported a problem, that looked like the given fixed bugs. The teacher created an exam, idled for serveral minutes and as he tried to start the exam his session was timed out.
Normally the saml ticket should be renewed in the background and everything should work, but in this case the ticket was not renewd and the session was unauthorized.

I opened the ticket @cust support to ask if also other customers reported such a behavior, as I do not realize that anymore @Linet.
It looks like a regression to me.
Comment 4 Dirk Schnick 2023-01-03 15:11:09 CET
Hi Florian, looks like I messed up my tickets/topics. This bug was opened caused by the log entries that looked not as a warning to me. Anyways this bug can be closed if it is only a warning. Sorry for my wrong post above.
Comment 5 Erik Damrose univentionstaff 2023-01-09 15:11:09 CET
management/univention-management-console/src/univention/management/console/base.py:406
It is just a warning and does not cause anything to fail at that point.