Bug 47693 - Support SSO-SAML authenticated usage of the HTTP-API-Import
Support SSO-SAML authenticated usage of the HTTP-API-Import
Status: NEW
Product: UCS@school
Classification: Unclassified
Component: UMC - CSV Import
UCS@school 4.4
All All
: P5 normal (vote)
: ---
Assigned To: UCS@school maintainers
:
: 47054 50012 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2018-08-29 10:00 CEST by Michael Grandjean
Modified: 2023-05-11 09:02 CEST (History)
8 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?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review: Yes
Ticket number: 2021100121000098
Bug group (optional): Usability
Max CVSS v3 score:


Attachments
schoolimport logfile (3.66 KB, text/plain)
2018-11-06 12:11 CET, Michael Grandjean
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Grandjean univentionstaff 2018-08-29 10:00:40 CEST
If a user is logged in via SAML, it is necessary to enter the user's password again to authenticate against the HTTP-API *every time* the "User import" module is opened. 
Especially when configuring / testing the import, this is very annoying (after a failed dry-run, it is necessary to close and re-open the module, afair).
Comment 1 Michael Grandjean univentionstaff 2018-11-02 17:08:33 CET
This still persists and is annoying af, especially since more and more school customers switch to SAML because of external services.

Today a customer had to enter their password four times to complete a single import run and download the password file (I witnessed this via TeamViewer):
1. Login UMC
2. Enter password again after opening the userimport module
3. Enter password again for the real run (after the dry-run)
4. Enter password again when clicking on "RELOAD" on the "Overview User Imports" grid (job was shown as "pending", although finished)
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2018-11-05 09:55:55 CET
(In reply to Michael Grandjean from comment #1)
> Today a customer had to enter their password four times to complete a single
> import run and download the password file (I witnessed this via TeamViewer):
> 1. Login UMC
> 2. Enter password again after opening the userimport module
> 3. Enter password again for the real run (after the dry-run)
> 4. Enter password again when clicking on "RELOAD" on the "Overview User
> Imports" grid (job was shown as "pending", although finished)

Despite the fact, that we should implement SAML, the behaviour looks broken. Afair the code should ask for the password in 2) and store it for 3) and 4) (at least if the same UMC process is used which has *not* been killed by the 10min timeout).
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2018-11-06 11:37:50 CET
(In reply to Sönke Schwardt-Krummrich from comment #2)
> Despite the fact, that we should implement SAML, the behaviour looks broken.
> Afair the code should ask for the password in 2) and store it for 3) and 4)
> (at least if the same UMC process is used which has *not* been killed by the
> 10min timeout).

I asked the UMC team, and they also think that the password should be saved after the first query. If that's not the case, it's really a bug. A fix for that would take some annoyance out of it, wouldn't it?
Comment 4 Michael Grandjean univentionstaff 2018-11-06 12:11:38 CET
Created attachment 9724 [details]
schoolimport logfile

Yes, it definitely would. However, I had a look at the logs again.

> (at least if the same UMC process is used which has *not* been killed by the 10min timeout).

That might be the culprit. But on the other hand, reviewing some users after the dry-run (why are there duplicates, are those that should be deleted correct ...?) and waiting for the import to finish can take more than 10 minutes each.
Comment 5 Michael Grandjean univentionstaff 2020-09-23 16:38:45 CEST
Still true, experienced this today on a customer system and it's so annoying ...
Comment 6 Erik Damrose univentionstaff 2020-09-23 17:33:05 CEST
An underlying UMC issue has been fixed with bug 50670, this may be an issue with this specific UMC module - may be worth re-investigating here
Comment 7 Dirk Schnick univentionstaff 2021-01-22 09:29:01 CET
Customer pointed on that issue during another ticket. It happens on the first opening of the module. He has to enter the credentials for 3 to 6 times; he said it depends how fast he enters the credentials. If fast he needs to enter 6 times; if slow 3 times.
Comment 8 Michael Grandjean univentionstaff 2021-03-24 11:15:58 CET
In some environments the UMC is now only accessible via SAML SSO, so the workaround to "not login with SSO" to avoid the descrobed behaviour is not available. This makes this bug even more annoying.
Comment 9 Dirk Schnick univentionstaff 2021-10-01 08:41:01 CEST
It looks like this bug is a duplicate to
https://forge.univention.org/bugzilla/show_bug.cgi?id=50012
I attached the actual ticket to both bugs. You have to enter the password multiple times if you use saml login.
Comment 10 Florian Best univentionstaff 2021-10-01 09:05:22 CEST
*** Bug 50012 has been marked as a duplicate of this bug. ***
Comment 12 Michael Grandjean univentionstaff 2021-11-24 14:15:29 CET
Unfortunately this is still broken and threatens project success. I will attach a video of the current behaviour in a fresh UCS@school 4.4v9 installation.
Comment 14 Ingo Steuwer univentionstaff 2022-01-04 13:04:07 CET
*** Bug 47054 has been marked as a duplicate of this bug. ***