Bug 51856 – Portal Preview App - Portal should re-render on login

Bug 51856 - Portal Preview App - Portal should re-render on login


Summary:	Portal Preview App - Portal should re-render on login

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	Portal
Version:	UCS 4.4
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.4-x
Assigned To:	Jürn Brodersen
QA Contact:	Dirk Wiesenthal

URL:
Keywords:

Depends on:
Blocks:	52314
	Show dependency tree / graph

Reported:	2020-08-20 12:10 CEST by Johannes Keiser
Modified:	2020-11-05 15:00 CET (History)
CC List:	2 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?:	1: Will affect a very few installed domains
How will those affected feel about the bug?:	2: A Pain – users won’t like this once they notice it
User Pain:	0.057
Enterprise Customer affected?:
School Customer affected?:	Yes
ISV affected?:
Waiting Support:	Yes
Flags outvoted (downgraded) after PO Review:
Ticket number:	2020081321000643
Bug group (optional):
Max CVSS v3 score:

Flags:	keiser: Patch_Available+

Attachments
Beginnings of a patch (not complete) (1.44 KB, patch) 2020-08-20 12:10 CEST, Johannes Keiser	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Johannes Keiser

2020-08-20 12:10:19 CEST

Created attachment 10462 [details]
Beginnings of a patch (not complete)

At the moment the Portal only re-render when the login button is clicked.
It should re-render when detecting a login instead.

Comment 1 Jürn Brodersen

2020-09-04 11:58:50 CEST

I was looking into the "window.postMessage" to communicate between the login and portal frames. But I decided against a direct communication for two reasons:
- It would not work with another IdP
- MDN recommends that both communication partner know the origin of each other. That would add another layer that needs to be configured in scenario that use a reverse proxy

I now check the login status in case the home tab is reselected and use the patch from Johannes to re-render the portal.

The patch can be found on the branch:
juern/4.4.4/portal_iframes

Comment 2 Dirk Wiesenthal

2020-09-08 09:57:42 CEST

App univention-portal=3.1.2

Comment 3 Dirk Wiesenthal

2020-09-08 09:58:58 CEST

Tested: Login in another tab. Login in another iframe. Logout in another tab. Logout in another iframe.

Code: OK
Login: OK

Comment 4 Dirk Wiesenthal

2020-09-22 21:40:23 CEST

Released version 3.1.2