Bug 42068 - univention-app-appliance sets notifyvender info wrong
univention-app-appliance sets notifyvender info wrong
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: General
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.1-3-errata
Assigned To: Erik Damrose
Alexander Kläser
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2016-08-19 16:04 CEST by Erik Damrose
Modified: 2016-09-21 21:01 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.034
Enterprise Customer affected?:
School Customer affected?:
ISV affected?: Yes
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Damrose univentionstaff 2016-08-19 16:04:42 CEST
univention-app-appliance sets the UCRv appliance/id/<appid>/notifyVendor wrong, because it tries to parse the ini value ApplianceNotifyVendor, when the correct ini value is NotifyVendor.

This causes the activation page to never send an installation notification, as only apps with NotifyVendor=True are reported
Comment 1 Erik Damrose univentionstaff 2016-08-19 16:13:58 CEST
Updated the value that is read from the INI to NotifyVendor

r71757 univention-app-appliance 0.2.2-36
r71758 merge to 4.2
r71759 yaml
Comment 2 Alexander Kläser univentionstaff 2016-08-23 17:10:47 CEST
Changes: OK, previous version always returned None for notifyVendor, after patch 
    this works fine now.
Merge to 4.2: OK
YAML file: OK, I renamed to package from univention-app-activation to 
    univention-app-appliance ;) [r71851]
Comment 3 Alexander Kläser univentionstaff 2016-08-23 17:34:46 CEST
BTW, is a release for 4.1-2 and 4.1-3 correct? If not, please adjust the YAML file.
Comment 4 Janek Walkenhorst univentionstaff 2016-08-24 12:06:46 CEST
<http://errata.software-univention.de/ucs/4.1/239.html>