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