URL:
https://github.com/SSSD/sssd/pull/109
Title: #109: SSSCTL: fix netgroup-show parsing
lslebodn commented:
"""
On (07/01/17 08:28), Jakub Hrozek wrote:
> On 6 Jan 2017, at 16:29, lslebodn <notifications(a)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.
OK
It was fixed in master by different commit and not by this PR. It was a
dependency for refactoring of nsss responder. And it's better to have the same
commit in older branches.
master:
* 7b293a5095ef3e63cd2e3f2ff01b7484bf6dcd38
sssd-1-14:
* c61f9fe1781fc4da2fc9e7f11071a4827cb0e463
LS
"""
See the full comment at
https://github.com/SSSD/sssd/pull/109#issuecomment-271513113