Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be
On 5 Jun 2018, at 09:16, fidencio <notifications(a)github.com>
I'll fire our internal CI and add the "Accepted" label after I get the
There's just one question, though:
IIRC we do the very same check for the well_known_sid in different parts of the code.
Wouldn't be worth (not for this PR) to have a new function doing the same thing all
over the place?
I was thinking about this as well, I think in some places where we only check for IPA SIDs
the check wouldn’t be needed but I think it wouldn’t hurt either.
See the full comment at https://github.com/SSSD/sssd/pull/590#issuecomment-394610890