Pavel, On Tue, Sep 27, 2016 at 1:25 PM, pbrezina <sssd-github-notificat...@fedorahosted.org> wrote: > URL: https://github.com/SSSD/sssd/pull/34 > Author: pbrezina > Title: #34: cache_req: move from switch to plugins > Action: opened > > PR body: > """ > cache_req grown quite big from the original code and it turned out > that using switch statements to branch code for different cases > makes the code quite hard to read and further extend and any > modification to the logic itself is difficult. > > This patch changes the switch statements to plugins with small > functions and separates logic into multiple modules. This gives > us better control over the code and improves readability and > maintainability while keeping code duplication to minimum. > > At the moment only cache req user by name, id and upn is > implemented as a proof of concept. If you like it, I will > finish the rest, it won't take much time. > """ >
+1 for the idea. Best Regards, -- Fabiano Fidêncio _______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
