Bug 51659 - redirected to "#module=....:undefined" when logging in with exactly 1 allowed UMC operation
redirected to "#module=....:undefined" when logging in with exactly 1 allowed...
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 4.4
Other Linux
: P5 normal (vote)
: UCS 5.0-0-errata
Assigned To: Florian Best
Sönke Schwardt-Krummrich
:
: 51657 (view as bug list)
Depends on:
Blocks: 53710
  Show dependency treegraph
 
Reported: 2020-07-10 12:39 CEST by Daniel Tröder
Modified: 2022-05-12 11:06 CEST (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?: 5: Blocking further progress on the daily work
User Pain: 0.429
Enterprise Customer affected?: Yes
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2020070921000546
Bug group (optional):
Max CVSS v3 score:


Attachments
umc-undefined-symptom-patch (632 bytes, patch)
2021-07-14 15:12 CEST, Julia Bremer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Tröder univentionstaff 2020-07-10 12:39:52 CEST
A staff user in a UCS@school system has exactly 1 allowed UMC operation "schoolimport-all" from the UMC policy "schoolimport-all".
After logging into the UMC, the user is redirected to "/univention/management/#module=schoolimport:undefined"
The user can enter the URL of the permitted UMC module "/univention/management/#module=schoolimport::0:" and the module opens successfully.
When closing it, a redirect to "/univention/management/#module=schoolimport:undefined" will happen though.

When another UMC operation
    (tested with "schoolgroups-workgroup" and "schoollists-all")
is added
    (to any UMC policy, tested with "ucsschool-umc-staff-default" or "schoolimport-all")
this does not happen, and the user gets both UMC modules listed.
Comment 1 Erik Damrose univentionstaff 2020-07-10 12:46:01 CEST
This caused bug 51657
Comment 2 Daniel Tröder univentionstaff 2020-07-13 10:46:08 CEST
*** Bug 51657 has been marked as a duplicate of this bug. ***
Comment 4 Nico Gulden univentionstaff 2020-12-01 13:20:22 CET
There has not been any recent activity on this bug. Has the problem been seen somewhere else as well in the meantime or has its assessment changed?
Comment 5 Christina Scheinig univentionstaff 2020-12-01 13:31:50 CET
I think we should check, if this is still an issue with the new school version instead of checking the Bug activity?

We implemented a workaround for the customers with this issue, and this is a configuration that might not be overwritten by a newer version. So could be still an issue.
Comment 6 Julia Bremer univentionstaff 2021-07-14 15:12:00 CEST
Created attachment 10786 [details]
umc-undefined-symptom-patch

The attached patch fixes this issue symptomatically, but not at its root.
Further research is needed.

The UMC Module flavor that is parsed from the stateHash is a String with the value 'undefined' instead of being of type undefined.
I wasn't yet able to determine why this is the case. 


The issue is reproducible in UCS with the UMC operationset diagnostic-all.
Comment 7 Sönke Schwardt-Krummrich univentionstaff 2021-08-27 11:43:47 CEST
Happened again with another customer.
Comment 8 Florian Best univentionstaff 2021-08-27 12:02:10 CEST
Caused by git:11078028f4055c2f25a916e23a922ce798a135f3 Bug #51185.

Fixed with:
univention-management-console.yaml
01b7a3d0ead0 | Bug #51659: fix UMC access if only one module without flavor is allowed to be accessed

univention-management-console (12.0.12-11)
01b7a3d0ead0 | Bug #51659: fix UMC access if only one module without flavor is allowed to be accessed
Comment 9 Sönke Schwardt-Krummrich univentionstaff 2021-08-31 12:56:08 CEST
OK: code change
OK: installation
OK: update
OK: changelog entry
~OK: advisory
OK: functional change
OK: package built and installable

Tested with 3 different users that were allowed to use 0, 1 or 2 UMC modules.
Looks fine to me.

Comitted small fix for advisory.
[5.0-0] 28e8f4834f Bug #51659: update YAML