URL:
https://github.com/SSSD/sssd/pull/449
Title: #449: cache: Check for max_id/min_id in cache_req
jhrozek commented:
"""
I'm sorry, but this patch is incorrect.
As I wrote in the ticket, you want to check the input id (data->id inside cache_req)
against the domain min/max limits inside the lookup_fn function of plugins that search by
ID.
My initial idea was to return a special error code (ERR_UID_OUTSIDE_RANGE?) so that the
lookup wouldn't even continue. But this would only handle requests where we actually
search the cache.
Alternatively, we could add another plugin method. Not sure which way is more preferred,
what do you think, @pbrezina
"""
See the full comment at
https://github.com/SSSD/sssd/pull/449#issuecomment-344212156