#3313 cache_req should use an negative cache entry for UPN based lookups
Closed: Fixed 7 years ago Opened 7 years ago 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 62df785 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

7 years ago

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

7 years ago

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)

7 years ago

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.

Login to comment on this ticket.

Metadata