Bug 52155 - Migrate univention-policy/python-lib/policy_result.py to python 3
Migrate univention-policy/python-lib/policy_result.py to python 3
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: univention-lib
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0
Assigned To: Florian Best
Dirk Wiesenthal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2020-09-29 14:48 CEST by Florian Best
Modified: 2021-05-25 16:01 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Development Internal
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

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2020-09-29 14:48:02 CEST
Migrate univention-policy/python-lib/policy_result.py to be python 3 compatible.
Comment 1 Florian Best univentionstaff 2020-09-30 13:01:08 CEST
Simple python fixes have been done.
We are decoding the whole output of `univention-policy-result` as UTF-8. In theory the LDAP schema could contain LDAP-syntaxes which are only ASCII or even binary output. In praxis, I don't find any.
If we would need this, we would need to change the API to an bytes interface and the consumers would have to decode it themselves.

univention-policy (11.0.1-1)
03d1402260a9 | Bug #52155: Migrate univention.lib.policy_result to Python 3

changelog-5.0-0.xml
03d1402260a9 | Bug #52155: Migrate univention.lib.policy_result to Python 3
Comment 2 Dirk Wiesenthal univentionstaff 2020-10-08 09:20:49 CEST
Code changes: OK
Some manual tests: OK
Chagelog: OK
Comment 3 Florian Best univentionstaff 2021-05-25 16:01:33 CEST
UCS 5.0 has been released:
 https://docs.software-univention.de/release-notes-5.0-0-en.html
 https://docs.software-univention.de/release-notes-5.0-0-de.html

If this error occurs again, please use "Clone This Bug".