On (28/06/16 18:50), Jakub Hrozek wrote:
On Mon, Feb 15, 2016 at 01:23:52PM +0100, Sumit Bose wrote:
On Wed, Jan 20, 2016 at 01:35:01PM +0100, Lukas Slebodnik wrote:
ehlo,
attached patch should fix ticket #2926
The default case (no adding, no removing services) is tested by pam-srv-tests.
If the patch is not ready for stable branch sssd-1-13 then we might merge this case with GPO code which uses slightly modified way using hash tables.
Hi Lukas,
sorry for the long delay. Yes, I think it would be a good idea to merge this case with the GPO code, i.e. to make ad_gpo_parse_map_option() and ad_gpo_parse_map_option_helper() some generic utility functions. This of course would make most of my comments below useless.
bye, Sumit
Do we still want to make this patch part of 1.14?
Don't worry it's on my todo list.
LS