Re: [SSSD] [PATCH] SDAP: Remove user from cache for missing user in LDAP

[Jakub Hrozek]

On Sun, Jun 21, 2015 at 11:21:50PM +0200, Michal Židek wrote:
>  On 06/19/2015 07:50 PM, Jakub Hrozek wrote:
>  >On Fri, Jun 19, 2015 at 06:44:25PM +0200, Lukas Slebodnik wrote:
>  >>On (19/06/15 18:03), Jakub Hrozek wrote:
>  >>>On Fri, Jun 19, 2015 at 05:59:04PM +0200, Michal Židek wrote:
>  >>>>On 06/18/2015 06:13 PM, Lukas Slebodnik wrote:
>  >>>>>ehlo,
>  >>>>>
>  >>>>>regression was cuased by patch
> d0b7e5fcfca7d0db9e3d19be7b51f34d03d3d720
>  >>>>>because function sysdb_get_real_name overrode returned code from
> LDAP.
>  >>>>>
>  >>>>>LS
>  >>>>>
>  >>>>>
>  >>>>
>  >>>>ACK.
>  >>>>
>  >>>>CI link:
>  >>>>http://sssd-ci.duckdns.org/logs/job/17/89/summary.html
>  >>>>
>  >>>>Michal
>  >>>
>  >>>Thanks.
>  >>>
>  >>>What about adding a default: to the switch? I know we currently can't
>  >>>get any other value, but it would be better to error out in that case.
>  >>Here you are.
>  >>
>  >>LS
>  >
>  >Thanks, this is what I wanted.
>  >
>  >ACK++
>  >
>  
>  Just for the record.
>  
>  CI for the new version:
>  http://sssd-ci.duckdns.org/logs/job/17/94/summary.html
>  
>  Tested and still works. I like the new version more. So
>  I need to give bigger ack than before:
>      _    ____ _  __
>     / \  / ___| |/ /
>    / _ \| |   | ' /
>   / ___ \ |___| . \
>  /_/   \_\____|_|\_\
>   
* master: 9fc96a4a2b07b92585b02dba161ab1eb2dbdad98
* sssd-1-12: d0d6956c3748ba670448edd548b8edd37c5f5c1a