#79 ldap driver needs a better way to fetch ldap results
Closed: Fixed None Opened 14 years ago by simo.

The current code implicitly assumes that only on ldap call is out on the wire at any given time.
This may not always be the case, if 2 calls are out at the same time ldap_result() as called by sdap_check_result() might actually swallow results from the other call which will have it's callback actually never called.

Need to change the code to have a single common fetcher that actually then notifies the waiting callbacks.


Fields changed

description: The current code implicitly assumes that only on ldap call is out on the wire at any given time.
This may not alwys be the case, if 2 calls are out at the same time ldap_result() as called by sdap_check_result() might actually swallow results from the other call which will have it's callback actually never called.

Need to checnge the code to have a single common fetcher that actually then notifies the waiting callbacks. => The current code implicitly assumes that only on ldap call is out on the wire at any given time.
This may not always be the case, if 2 calls are out at the same time ldap_result() as called by sdap_check_result() might actually swallow results from the other call which will have it's callback actually never called.

Need to change the code to have a single common fetcher that actually then notifies the waiting callbacks.

Fields changed

fixedin: => 0.5.0
resolution: => fixed
status: new => closed

Fields changed

rhbz: => 0

Metadata Update from @simo:
- Issue assigned to simo
- Issue set to the milestone: SSSD 1.0

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/1121

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