Bug 52466 - Backport memory leak/usage fix in python-pysaml2
Backport memory leak/usage fix in python-pysaml2
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 4.4
Other Linux
: P5 normal (vote)
: UCS 4.4-7-errata
Assigned To: Florian Best
Jürn Brodersen
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2020-12-08 00:47 CET by Florian Best
Modified: 2020-12-16 16:03 CET (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, SAML, UCS Performance
Max CVSS v3 score:
best: Patch_Available+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2020-12-08 00:47:30 CET
NameID instances are very large and hold a lot of XML element tree object references. This doesn't scale for a larger amount of sessions.

Instead of storing the NameID instances, a serialized string can be used.

We should backport the corresponding patch:

https://github.com/IdentityPython/pysaml2/commit/fbb0b5a1e76ff2e638dafcbd8cf6e730d1d68e0a
Comment 1 Florian Best univentionstaff 2020-12-08 00:55:14 CET
Fixed in:

r19233 | Bug #52466: add 004-nameid-memory-consumption.quilt

python-pysaml2.yaml
717ca8fced64 | YAML Bug #52466
Comment 2 Jürn Brodersen univentionstaff 2020-12-10 10:29:46 CET
What I tested:
Upgrade -> OK
Login -> OK
Login SAML -> OK
users/user -> OK
portal login -> OK
portal edit -> OK
jenkins -> OK
Merge 5.0 -> Fixed in upstream -> OK
YAML -> OK

I rebuild the package because the version number was lower than the version of the old package