Bug 50359 – Appliances don't have customized Portals

Bug 50359 - Appliances don't have customized Portals


Summary:	Appliances don't have customized Portals

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	App Center
Version:	UCS 4.4
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.4-2-errata
Assigned To:	Jannik Ahlers
QA Contact:	Dirk Wiesenthal

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2019-10-14 11:52 CEST by Jannik Ahlers
Modified:	2019-10-17 18:43 CEST (History)
CC List:	2 users (show)

See Also:	47079
What kind of report is it?:	Bug Report
What type of bug is this?:	1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?:	3: Will affect average number of installed domains
How will those affected feel about the bug?:	1: Nuisance – not a big deal but noticeable
User Pain:	0.017
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):	Appliance
Max CVSS v3 score:

Attachments
patch (648 bytes, patch) 2019-10-14 15:34 CEST, Jannik Ahlers	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 Jannik Ahlers

2019-10-14 11:52:37 CEST

When configured in the provider portal, app appliances should have a portal with custom title, logo, background, and text color (app ini values: AppliancePortalLogo, AppliancePortalCssBackground, AppliancePortalTitle, AppliancePortalFontColor).
This is not working for the currently released appliances and for appliances that are being built right now.

Maybe this has something to do with it:
utils/base_appliance.sh:        /usr/sbin/univention-app-appliance --not-configure-portal $main_app

Comment 1 Erik Damrose

2019-10-14 15:08:04 CEST

The package univention-app-appliance provides a script to configure the portal:

usr/lib/univention-system-setup/appliance-hooks.d/30_adjust_portal:/usr/sbin/univention-app-appliance --only-configure-portal $(ucr get umc/web/appliance/id)

Output from that script should be available in /var/log/univention/setup.log

Comment 2 Jannik Ahlers

2019-10-14 15:34:01 CEST

Created attachment 10204 [details]
patch

/usr/sbin/univention-app-appliance --only-configure-portal $(ucr get umc/web/appliance/id) fails with:

Traceback (most recent call last):
  File "/usr/sbin/univention-app-appliance", line 309, in <module>
    configure_portal(app)
  File "/usr/sbin/univention-app-appliance", line 88, in configure_portal
    result = module.lookup(None, lo, 'cn=domain', base=UCR['ldap/base'], scope='sub')
AttributeError: 'NoneType' object has no attribute 'lookup'

The reason for this is that the udm modules aren't loaded.

Comment 3 Jannik Ahlers

2019-10-17 13:29:12 CEST

I applied the patch.

f049f5b3db Bug #50359: yaml
735fb1d419 Bug #50359: fix appliance portal branding
2a7918b526 Bug #50359: code cleanup

Successful build
Package: univention-app-appliance
Version: 3.0.1-2A~4.4.0.201910171109
Branch: ucs_4.4-0
Scope: errata4.4-2

Comment 4 Dirk Wiesenthal

2019-10-17 16:38:11 CEST

Portal: OK
Code: OK
YAML: OK

Comment 5 Erik Damrose

2019-10-17 18:43:33 CEST

Released as unmaintained errata for 4.4-2. I removed the yaml file from staging.