Bug 39226 - Make AppCenter module SAML ready
Make AppCenter module SAML ready
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: SAML
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.1
Assigned To: Florian Best
Erik Damrose
: interim-1
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-08-20 10:18 CEST by Florian Best
Modified: 2015-11-17 12:11 CET (History)
1 user (show)

See Also:
What kind of report is it?: ---
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

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2015-08-20 10:18:13 CEST
The appcenter module should be adjusted to work with the new SAML authentication.

It currently establishes a UMC connection to another host (to install apps there).
The authentication type should be passed along with the authentication.
Comment 1 Florian Best univentionstaff 2015-08-20 14:52:52 CEST
Hm. The joinscripts are also called with the password.
Comment 2 Florian Best univentionstaff 2015-09-22 12:10:32 CEST
If logged in via single sign on the password is asked before installing an application.

univention-management-console-module-appcenter (5.0.5-1):
r63874 | Bug #39226: require a non-SSO session to install apps
Comment 3 Erik Damrose univentionstaff 2015-09-30 13:17:06 CEST
Reopen:
If i try to install an app, The password dialog is shown. After that, i have to click on install again. The action that required the password dialog should be executed after the credentials have been supplied.

Missing changelog entry
Comment 4 Florian Best univentionstaff 2015-10-05 17:51:19 CEST
(In reply to Erik Damrose from comment #3)
> Reopen:
> If i try to install an app, The password dialog is shown. After that, i have
> to click on install again. The action that required the password dialog
> should be executed after the credentials have been supplied.
Yes, I adapted the behavior such that it's not necessary to click install again.

> Missing changelog entry
added.
Comment 5 Erik Damrose univentionstaff 2015-10-06 13:29:49 CEST
OK: App installation + deinstallation requires password
OK: Integration / user presentation
OK: changelog
Comment 6 Erik Damrose univentionstaff 2015-11-09 15:04:09 CET
The issue mentioned at comment 3 has reappeared:
Reopen:
If i try to install an app, The password dialog is shown. After that, i have to click on install again. The action that required the password dialog should be executed after the credentials have been supplied.
Comment 7 Florian Best univentionstaff 2015-11-09 15:31:33 CET
(In reply to Erik Damrose from comment #6)
> The issue mentioned at comment 3 has reappeared:
> Reopen:
> If i try to install an app, The password dialog is shown. After that, i have
> to click on install again. The action that required the password dialog
> should be executed after the credentials have been supplied.
yep, I renamed a internal function and accidentally didn't fix one reference to it.
Comment 8 Erik Damrose univentionstaff 2015-11-10 10:47:55 CET
OK, works as before. Verified
Comment 9 Stefan Gohmann univentionstaff 2015-11-17 12:11:45 CET
UCS 4.1 has been released:
 https://docs.software-univention.de/release-notes-4.1-0-en.html
 https://docs.software-univention.de/release-notes-4.1-0-de.html

If this error occurs again, please use "Clone This Bug".