URL:
https://github.com/SSSD/sssd/pull/590
Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be
resolved
jhrozek commented:
"""
On 5 Jun 2018, at 09:16, fidencio <notifications(a)github.com>
wrote:
Ack!
I'll fire our internal CI and add the "Accepted" label after I get the
results.
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