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

Michal Židek mzidek at redhat.com
Sun Jun 21 21:21:50 UTC 2015


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:
     _    ____ _  __
    / \  / ___| |/ /
   / _ \| |   | ' /
  / ___ \ |___| . \
/_/   \_\____|_|\_\



Michal




More information about the sssd-devel mailing list