Bug 51764 – SAML login page won't load in UCS 5 portal preview due to "X-Frame-Options: SAMEORIGIN"

Bug 51764 - SAML login page won't load in UCS 5 portal preview due to "X-Frame-Options: SAMEORIGIN"


Summary:	SAML login page won't load in UCS 5 portal preview due to "X-Frame-Options: S...

Status:	NEW

Product:	UCS
Classification:	Unclassified
Component:	SAML
Version:	UCS 4.4
Hardware:	Other Windows NT

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	UCS maintainers
QA Contact:	UCS maintainers

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2020-08-03 12:19 CEST by Moritz Bunkus
Modified:	2020-10-12 12:08 CEST (History)
CC List:	2 users (show)

See Also:	51211
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
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 Moritz Bunkus 2020-08-03 12:19:12 CEST

The package univention-saml contains the file /usr/share/simplesamlphp/modules/univentiontheme/themes/univention/default/includes/header.php which unconditionally sets the "X-Frame-Options" header to "SAMEORIGIN". This is a problem when trying to use the UCS 5 portal preview (on UCS 4.4).

For example, if the portal is available at https://portal.my.site and the SSO login URL is https://ucs-sso.my.site, the browser will refuse to display content from https://ucs-sso.my.site in the frame loaded from https://portal.my.site, making login impossible.

Commenting out the line setting said header restores functionality.

There are several UCR variables that need to be set for the UCS5 portal preview to work correctly as far as I know. They're all set. Making this header depend on one of those variables might be a solution, as might be introducing a new UCR variable solely for this purpose.

Comment 1 Florian Best

2020-08-03 13:15:06 CEST

Hm, I thoght it was removed in Bug #51211 but seem to get re-added with git:bf86f1f2467ab92ae394f94fac7eac72cdafed31.

Comment 2 linux@osit.cc 2020-10-12 12:08:35 CEST

Same here.