URL:
https://github.com/SSSD/sssd/pull/152
Title: #152: Add a tevent wrapper around libcurl's asynchronous interface
lslebodn commented:
"""
On (01/03/17 04:41), Pavel Březina wrote:
pbrezina commented on this pull request.
> + urls[i],
+ headers,
+ inbufs[i],
+ 5);
+ if (ctx == NULL) {
+ DEBUG(SSSDBG_FATAL_FAILURE, "Could not create request\n");
+ talloc_zfree(tool_ctx);
+ return 1;
+ }
+ tevent_req_set_callback(req, request_done, tool_ctx);
+ }
+
+ while (tool_ctx->done == false) {
+ tevent_loop_once(ev);
+ }
+
We should not create a deadlock, if there is one its a bug :-)
Yes; but in case of bug in libcurl/somewhere else we should fail after timeout
and exit with non-zere value.
It's more friednly for tests rather then waiting for unfinished
requests.
LS
"""
See the full comment at
https://github.com/SSSD/sssd/pull/152#issuecomment-283330883