Bug 52345 - Fetch portal.json only once if not necessary to do a polling
Summary: Fetch portal.json only once if not necessary to do a polling
Status: CLOSED FIXED
Alias: None
Product: UCS
Classification: Unclassified
Component: Portal
Version: UCS 4.4
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 4.4-6-errata
Assignee: Florian Best
QA Contact: Dirk Wiesenthal
URL:
Keywords:
Depends on: 52314
Blocks:
  Show dependency tree / graph
 
Reported: 2020-11-10 19:52 CET by Florian Best
Modified: 2020-11-11 17:03 CET (History)
4 users (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):
Customer ID:
Max CVSS v3 score:
best: Patch_Available+

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 2020-11-10 19:52:06 CET 
The portal.json is fetched multiple (up to 15 - 20) times in various situations.
It should be fetched only once.
A deferred should be added which blocks other requests while a request is already ongoing.

The original reason to fetch it so often is in edit mode so after modifying we give the listener module enough time to write the new entries so that we know the latest state. This was originally implemented in git:f5adc6cc7d8d526fcb5af6bad222e8ec4299d3b8.

+++ This bug was initially created as a clone of Bug #52314 +++
Comment 1 Florian Best univentionstaff 2020-11-10 20:03:12 CET 
Cherry-picked the patch.

univention-portal.yaml
1b3016690f46 | YAML Bug #52345

univention-portal (3.0.2-10)
662a88f20641 | Bug #52345: do not fetch portal.json multiple times
Comment 2 Dirk Wiesenthal univentionstaff 2020-11-11 13:49:22 CET 
OK: portal.json is only fetched once
OK: YAML