Bug 53853 - Portal uses blocking synchronous HTTP requests to UMC-Webserver
Summary: Portal uses blocking synchronous HTTP requests to UMC-Webserver
Status: CLOSED FIXED
Alias: None
Product: UCS
Classification: Unclassified
Component: Portal
Version: UCS 5.0
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 5.0-1-errata
Assignee: Florian Best
QA Contact: Dirk Wiesenthal
URL: https://git.knut.univention.de/univen...
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-30 12:20 CEST by Florian Best
Modified: 2022-06-08 16:50 CEST (History)
0 users

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): Large environments, UCS Performance
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 Florian Best univentionstaff 2021-09-30 12:20:00 CEST 
Every time a portal.json is fetched (aka every time accessing the Portal) a synchronous HTTP request is done to the UMC-Webserver which blocks the main thread and therefore all other concurrent requests. We should switch to a asynchronous HTTP client.
Comment 2 Florian Best univentionstaff 2022-05-03 12:40:14 CEST 
Merged by Dirk → RESOLVED FIXED:

univention-portal (4.0.5-77)
fe7778f3c5a3 | Bug #53853: fix[portal]: make unittests asynchronous
08eaaf3be87a | Bug #53853: fix[portal]: fetch UMC session-info asynchronously
86ef8af1f71e | Bug #53853: fix[portal]: make request methods asyncronous
7f88d7b94d52 | Bug #53853: fix[portal]: lower case group names as is_member_of() checks for lowercased groups
c40c72d64b77 | Bug #53853: fix[portal]: fix LDAP DN escaping
86bce8aa55fb | Bug #53853: fix[portal]: fix invalid URL Path in Gateway
Comment 3 Dirk Wiesenthal univentionstaff 2022-06-08 13:06:57 CEST 
YAML: OK
Code review: OK
Test for regressions: OK, none found