URL: https://github.com/SSSD/sssd/pull/187
Title: #187: Add support to lookup for users/groups in subdomains just by the 
user shortname

pbrezina commented:
"""
Hmm, I see more problems here. Domains can be actively enabled or disabled now 
from the data provider. The domains in lookup order list do not reflect this. I 
think the best thing to do is to create new list of domains that would contain 
pointer to the sss_domain_info structure. Maybe, we can move the code to 
cache_req since it is the only consumer.

```c
struct cache_req_domain {
    struct sss_domain_info *domain;
    struct cache_req_domain *prev;
    struct cache_req_domain *next;
}
```

This will make sure that all changes done to the domain a reflected during the 
search.

As I read the code, the subdomains that were removed from the trust setup are 
only disabled, not removed from the list. We can use this so we don't have to 
built the list always from scratch, i.e. only add new domains into the list (to 
the end of the list or after its parent domain?).

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/187#issuecomment-288693368
_______________________________________________
sssd-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to