Univention Bugzilla – Full Text Bug Listing |
Summary: | Traceback with cross-school users after being removed from a school (4.2) | ||
---|---|---|---|
Product: | UCS | Reporter: | Valentin Heidelberger <heidelberger> |
Component: | S4 Connector | Assignee: | Arvid Requate <requate> |
Status: | CLOSED FIXED | QA Contact: | Felix Botner <botner> |
Severity: | normal | ||
Priority: | P5 | CC: | gohmann, markus.daehlmann, michelsmidt, requate, scheinig, schwardt |
Version: | UCS 4.2 | ||
Target Milestone: | UCS 4.2-4-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
See Also: | https://forge.univention.org/bugzilla/show_bug.cgi?id=49850 | ||
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?: | 2: Will only affect a few installed domains | How will those affected feel about the bug?: | 3: A User would likely not purchase the product |
User Pain: | 0.171 | Enterprise Customer affected?: | |
School Customer affected?: | Yes | ISV affected?: | |
Waiting Support: | Yes | Flags outvoted (downgraded) after PO Review: | |
Ticket number: | 2018031621000473 | Bug group (optional): | |
Max CVSS v3 score: | |||
Bug Depends on: | 25709, 46682, 47636 | ||
Bug Blocks: | 46692, 47104 | ||
Attachments: | bug46971-reproducer1.sh |
Description
Valentin Heidelberger
2018-05-07 16:19:20 CEST
* Patch backported to branch 4.2-4 * Package imported and built in errata4.2-4 * d25f6c0f33 | Advisory dont think that helps, object_memberships_sync_to_ucs already checks the sync mode i could reproduce the traceback with * create users test1, test2 * add test1 to domain admins * add test2 to domain admins (separate step ! ) * create another user test3 to trigger the group_mapping_cache_ucs ( now the cache contains at least test1 and test2) * remove test1, the group_mapping_cache_ucs is not updated and still contains test1 * add test1 as member to domain admins (via ldapvi) to simulate the ucssschool behavior i think we should just remove the "object['dn'].lower()" from self.group_mapping_cache_ucs in sync_to_ucs() if property_type is user and object['modtype'] is delete Created attachment 9539 [details] bug46971-reproducer1.sh Ok, when objects are deleted or moved remove the their DN from both group member mapping caches. 147232dc33 | Code cleanup: Improve readability 92f8e177e9 | Code cleanup: Improve readability 035dbabe63 | Fix traceback c2985a6d52 | Changelog 6b3ee3f602 | Advisory I fixed a code comment: 7b9fef72c9 | Fix code comment 0e44caa94d | Changelog 528d008c4a | Advisory OK - code cleanup OK - fixed traceback OK - manual group sync tests OK - s4connector jenkins tests (4.2-4) OK - yaml |