Bug 50118 - [UDM HTTP API] split up source code
Summary: [UDM HTTP API] split up source code
Status: REOPENED
Alias: None
Product: UCS
Classification: Unclassified
Component: UDM - REST API
Version: UCS 4.4
Hardware: Other Linux
: P5 normal
Target Milestone: ---
Assignee: UMC maintainers
QA Contact: UMC maintainers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-09-06 04:19 CEST by Daniel Tröder
Modified: 2025-04-04 09:20 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.171
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Customer ID:
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 Daniel Tröder univentionstaff 2019-09-06 04:19:30 CEST 
The current state of the source code of the UDM REST API (one file with 3700 lines) makes it very difficult to

* reuse code
* review code
* track changes

All three points are very important for the maintenance of UCS.

Split the code into multiple files along lines that facilitate the above points.
Comment 1 Jan-Luca Kiok univentionstaff 2024-06-27 12:10:38 CEST 
This issue has been filed against UCS 4.4.

UCS 4.4 is out of general maintenance and components may have vastly changed in later releases. Thus, this issue is now being closed.

If this issue still occurs in newer versions, please use "Clone this bug" or reopen this issue. In this case please provide detailed information on how this issue is affecting you.
Comment 2 Daniel Tröder univentionstaff 2024-06-27 13:50:32 CEST 
Still required for maintenance.
Comment 3 Florian Best univentionstaff 2025-04-04 09:20:55 CEST 
Situation already improved a bit:

univention/admin/rest/shared_memory.py
univention/admin/rest/module.py
univention/admin/rest/async_client.py
univention/admin/rest/client/__init__.py
univention/admin/rest/client/__main__.py
univention/admin/rest/openapi.py
univention/admin/rest/hal.py
univention/admin/rest/server/__init__.py
univention/admin/rest/server/__main__.py
univention/admin/rest/html_ui.py
univention/admin/rest/http_conditional.py
univention/admin/rest/__init__.py
univention/admin/rest/sanitizer.py
univention/admin/rest/ldap_connection.py
univention/admin/rest/utils.py
univention/admin/rest/__main__.py