URL: https://github.com/SSSD/sssd/pull/551 Title: #551: GPO: Fix bug with empty GPO rules
rdratlos commented: """ Hi Michal,, of course your PR should continue to be reviewed. My intention was just to present an alternative approach based on some analysis half a year ago that guided me to SID merging. After some further testing and analysis today, I would recommend to fix issue [#3680](https://pagure.io/SSSD/sssd/issue/3680) based on your proposal. But what I'd like to propose is to add an explanation and references to function ad_gpo_cse_done() how GPO settings are to be merged and overridden. This should prevent a third or fourth contributor showing up and wanting to implement SID merging :-). Kind regards Thomas """ See the full comment at https://github.com/SSSD/sssd/pull/551#issuecomment-382187714
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org