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?