Bug 39395 - univention-system-activation: missing uuid in appcenter notification
univention-system-activation: missing uuid in appcenter notification
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: System setup
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.0-3-errata
Assigned To: Felix Botner
Florian Best
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-09-23 17:13 CEST by Felix Botner
Modified: 2015-09-24 14:38 CEST (History)
2 users (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
wsgi.py.patch (1.40 KB, patch)
2015-09-23 17:14 CEST, Felix Botner
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Botner univentionstaff 2015-09-23 17:13:45 CEST
This is the request from my test system

Request URL:https://appcenter.software-univention.de/postinst?app=owncloud8&version=8.0.4-1&uuid=&action=install&status=200&role=domaincontroller_master
Request Method:GET
Status Code:400 Bad Request
Date:Wed, 23 Sep 2015 15:54:23 GMT


uuid is missing

because univention-system-activation/src/wsgi.py imports the license, reloads ucr and gets the uuid/license from ucr. But uuid/license is set by a listener module, and this module is maybe not yet finished at this point.


Added a patch to read the uuid from the license file.
Comment 1 Felix Botner univentionstaff 2015-09-23 17:14:33 CEST
Created attachment 7183 [details]
wsgi.py.patch
Comment 2 Felix Botner univentionstaff 2015-09-23 17:16:01 CEST
I have tested this by copying the file to an unactivated univention app and then continued with the activation.
Comment 3 Felix Botner univentionstaff 2015-09-24 09:39:02 CEST
updated univention-system-activation in errata4.0-3, merged to 4.1-0

The default UUID is now '00000000-0000-0000-0000-000000000000' (as in the appcenter notification in univention-management-console-module-appcenter).

YAML: 2015-09-08-univention-system-activation.yaml
Comment 4 Florian Best univentionstaff 2015-09-24 12:08:41 CEST
Changes are working. YAML is OK. I build the package for 4.1.
Comment 5 Janek Walkenhorst univentionstaff 2015-09-24 14:38:05 CEST
<http://errata.software-univention.de/ucs/4.0/334.html>