Univention Bugzilla – Bug 34507
Clean up useless cache_entry "security-copy" code in univention-directory-listener
Last modified: 2016-07-21 15:16:03 CEST
Since r28653 of Bug #24273 every handler already gets it own private copy, so there's no longer the need for the univention-directory-listener to generate a copy of the new cache_entry before passing the data to handlers_update.
This "security-copy" of the new (in memory) cache entry was introduced for Bug #21914, but that change did not isolate the modules from each other, it only protected the cache from rogue modules.
We should remove the "security-copy" code introduced via Bug #21914.
+++ This bug was initially created as a clone of Bug #34499 +++
Created attachment 5862 [details]
Untested code removal
FYI: through handler_exec -> handlers_argtuple -> handlers_entrydict each handler gets an individual copy of the old|new dictionaries, so it's guaranteed that no communication between the modules is possible.
r69897 | Bug #34507 UDL: Remove 'security-copy'
r69896 | Bug #34507 UDL: Fix wrong error message
r69910 | Bug #22383,Bug #30227,Bug #30263,Bug #34324,Bug #34507,Bug #34738,Bug #3490,Bug #38696,Bug #39509,Bug #40600,Bug #41261: UDL YAML
Code review: Ok