URL:
https://github.com/SSSD/sssd/pull/776
Title: #776: NEGCACHE: repopulate negative cache after get_domains and initialize UPN
negative cache as well
jhrozek commented:
"""
I have two more comments:
1. (minor) there is a comment ` /* All domains were local */` which I think is confusing.
This part of code is executed when all non-local domains were processed, right? Do you
think it would improve the readability of the code to fix the comment to say that there
are no more non-local domains to try?
2. Given that `sss_dp_get_domains_process` is executed periodically, do you think the
negative cache would grow with every execution? Or do you know if tdb is smart enough to
only add one key-value pair?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/776#issuecomment-472405509