On 22 Sep 2017, at 11:10, Sumit Bose <sbose(a)redhat.com> wrote:
> - Add support for ActiveDirectory's logonHorous restrictions
> (
https://github.com/SSSD/sssd/pull/269)
> This PR comes from an external contributor and as far as I
> understood there's still some work to be done. So, should be postponed
> to the next release
I think it can be postponed. Nevertheless, unfortunately we are giving
mixed messages to the contributor about where to code should be placed.
Jakub and Simo preferred to move it to the AD provider because the
attribute is basically used only by AD. I on the other hand suggested to
keep it in the general LDAP code together with other AD related legacy
options.
So to get further here I think we should agree on what we want first.
I forgot/didn’t have bandwidth to reply to the PR but I’m fine with your approach as well,
because keeping the code locality makes sense.