URL: https://github.com/SSSD/sssd/pull/109 Title: #109: SSSCTL: fix netgroup-show parsing
jhrozek commented: """ > On 6 Jan 2017, at 16:29, lslebodn <notificati...@github.com> wrote: > > On (06/01/17 05:52), Jakub Hrozek wrote: > >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 > > > Do you think that it would be a problem to backport > 7b293a5095ef3e63cd2e3f2ff01b7484bf6dcd38 into 1.14 > rather that this patch? > > Upstream integration tests passed with it. I haven't tried downstrem > tests. > That would also work. > LS > — > 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-270927695>, or mute the > thread > <https://github.com/notifications/unsubscribe-auth/AArrAjMI-4Ngp0iWszBelCCTnwc6bLh3ks5rPl3dgaJpZM4LMfe2>. > """ See the full comment at https://github.com/SSSD/sssd/pull/109#issuecomment-271093554
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
