Bug 52723 - Wrong status code for get template in OpenAPI schema of UDM REST API
Wrong status code for get template in OpenAPI schema of UDM REST API
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UDM - REST API
UCS 4.4
Other Linux
: P5 normal (vote)
: UCS 4.4-7-errata
Assigned To: Daniel Tröder
Florian Best
:
Depends on:
Blocks: 50974 52769
  Show dependency treegraph
 
Reported: 2021-01-29 18:03 CET by Daniel Tröder
Modified: 2021-02-11 12:08 CET (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?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 3: A User would likely not purchase the product
User Pain: 0.051
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:
troeder: Patch_Available+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Tröder univentionstaff 2021-01-29 18:03:26 CET
The OpenAPI schema of the UDM REST API says that it will return HTTP 201 for a new object template, but it returns HTTP 200.

This breaks the autogenerated REST client for the UDM REST API client.
Comment 1 Daniel Tröder univentionstaff 2021-01-29 18:05:30 CET
Fixed in branch dtroeder/52723_http_status_object_template

[68ee5fb329] Bug #52723: fix status code for UDM REST API object template

https://git.knut.univention.de/univention/ucs/-/merge_requests/62
Comment 2 Daniel Tröder univentionstaff 2021-02-03 15:43:23 CET
The OpenAPI schema of the UDM REST API now returns "200" as status code for the retrieval of object creation templates.

[4.4-7 f19af2d6a6] Bug #52723: return HTTP 200 when an object creation template was requested
[4.4-7 784dcb99da] Bug #52723: advisory
[4.4-7 dbc27409bd] Bug #52723: fix typo

univention-directory-manager-rest (9.0.16-12)
Comment 3 Daniel Tröder univentionstaff 2021-02-03 16:01:52 CET
Merge request to 5.0-0: https://git.knut.univention.de/univention/ucs/-/merge_requests/66
Comment 4 Florian Best univentionstaff 2021-02-05 14:23:41 CET
OK: schema returns 200 instead of 201
OK: YAML
OK: MR exists (needs rebase)
Comment 5 Daniel Tröder univentionstaff 2021-02-08 09:15:09 CET
(In reply to Florian Best from comment #4)
> OK: MR exists (needs rebase)
Rebase done.