#3313 cache_req should use an negative cache entry for UPN based lookups
Closed: Fixed Opened by sbose.

In the old (non-cache_req) code a special name was used to add UPN lookups to the negative cache because otherwise the UPN lookup wasn't done at all because the previous plain name based lookup would already add an entry to the negative cache. See commit 62df78512145db94b51c5573d4df1737197e368a for details.

Please see attached quick-and-dirty patch I used to get around this. There might be a more elegant solution.


Patch looks good. I think it would be better to extract it into sss_ncache_set_upn(), sss_ncache_check_upn().

Fields changed

owner: somebody => sbose
patch: 0 => 1
status: new => assigned

Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.15.1

Metadata Update from @sbose:
- Issue assigned to sbose
- Issue set to the milestone: SSSD 1.15.1

master:

  • 54039570d26e29444c398aa4ad6ba638f1713566

Metadata Update from @lslebodn:
- Custom field design_review reset
- Custom field mark reset
- Custom field patch adjusted to on (was: 1)
- Custom field review reset
- Custom field sensitive reset
- Custom field testsupdated reset
- Issue close_status updated to: None

Metadata Update from @lslebodn:
- Custom field design_review reset
- Custom field mark reset
- Custom field review reset
- Custom field sensitive reset
- Custom field testsupdated reset
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/4346

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata