On 20.01.2016 10:52, Fraser Tweedale wrote:
Thank you, patch works for me. However, I'm not sure where the original
error was located, it looked like something in _httplib_request doesn't
work properly with SSL. Your patch uses _httplib_request without TLS so
it should work.
On Wed, Jan 20, 2016 at 09:30:29AM +0100, Martin Kosek wrote:
On 01/20/2016 08:45 AM, Fraser Tweedale wrote:
The attached patch removes a workaround introduced as part of
Alternatively, if we want to keep the "workaround" I will submit a
different patch that removes unused code and FIXME comments :)
You may also want to check FreeIPA spec file, if there is now no extra curl
dependency. I would leave it up to Martin Basti, to confirm that the original
issue cannot appear again. It was a nightmare to troubleshoot, as I heard :)
Good pickup on the curl dependency; indeed it is no longer needed.
Updated patch attached.
I would like to push this patch only to master, as the issue before
wasn't regularly reproducible, and I will keep eye on it.
Also I will remove the ticket #4676 from description, because ticket has
been closed in 4.1 Milestone.
ACK with keeping eyes on it
Pushed to master: fd7ea2c9395651d5bce41cc603557fea107f65a7
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code