URL: https://github.com/SSSD/sssd/pull/109
Title: #109: SSSCTL: fix netgroup-show parsing

jhrozek commented:
"""
This patch is OK, but only for sssd-1-14. In master, we already fall back to 
parsing the name string as short name if parsing the qualified name fails. I’m 
not thrilled about that, because it can conceal legitimate errors, but it’s 
needed atm to use cache_req everywhere.

So yeah, ack to this patch for sssd-1-14

> On 6 Jan 2017, at 10:59, lslebodn <notificati...@github.com> wrote:
> 
> @jhrozek <https://github.com/jhrozek> Have you already decided whether it 
> would be better to have this patch or 7b293a5 
> <https://github.com/SSSD/sssd/commit/7b293a5095ef3e63cd2e3f2ff01b7484bf6dcd38>?
> 
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub 
> <https://github.com/SSSD/sssd/pull/109#issuecomment-270870268>, or mute the 
> thread 
> <https://github.com/notifications/unsubscribe-auth/AArrAj2dX_wtSbPfE5pfLpRJ1ApbWU7wks5rPhCMgaJpZM4LMfe2>.
> 


"""

See the full comment at 
https://github.com/SSSD/sssd/pull/109#issuecomment-270906940
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

Reply via email to