URL:
https://github.com/SSSD/sssd/pull/868
Title: #868: ifp: let cache_req parse input name so it can fallback to upn search
pbrezina commented:
"""
I am not entirely sure about this change:
```diff
/* IFP serves both POSIX and application domains. Requests that need
* to differentiate between the two must be qualified
*/
- subreq = cache_req_send(state, state->rctx->ev, state->rctx,
- state->ncache, 0,
- CACHE_REQ_ANY_DOM,
- state->domname, data);
+ subreq = cache_req_send(state, state->rctx->ev, state->rctx,
state->ncache,
+ 0, CACHE_REQ_ANY_DOM, NULL, data);
if (subreq == NULL) {
- tevent_req_error(req, ENOMEM);
- return;
+ ret = ENOMEM;
+ goto done;
}
`cache_req` internally will parse the input name and if it is fully qualified it will
search only in the right domain. @jhrozek Is there any specific reason why we would have
to provide search domain already in this `cache_req` call?
```
"""
See the full comment at
https://github.com/SSSD/sssd/pull/868#issuecomment-522561558