[PATCH] Call ldap_install_tls() on ldaps connections

[Sumit Bose]

Hi,

the ldap_init_fd() patch introduced another regression with ldaps URIs.
This patch should fix it.

bye,
Sumit

[Stephen Gallagher]

On Tue, 2011-07-05 at 14:19 +0200, Sumit Bose wrote:
>  Hi,
>  
>  the ldap_init_fd() patch introduced another regression with ldaps URIs.
>  This patch should fix it. 
The patch looks good. It's a bit unfortunate that it's a synchronous
action, but we can resolve that later. Please open an upstream ticket to
have us handle the SSL connection ourselves.

I'm acking the patch, contingent upon the regression test suite
completing successfully.

[Stephen Gallagher]

On Tue, 2011-07-05 at 09:41 -0400, Stephen Gallagher wrote:
>  On Tue, 2011-07-05 at 14:19 +0200, Sumit Bose wrote:
>  > Hi,
>  > 
>  > the ldap_init_fd() patch introduced another regression with ldaps URIs.
>  > This patch should fix it.
>  
>  The patch looks good. It's a bit unfortunate that it's a synchronous
>  action, but we can resolve that later. Please open an upstream ticket to
>  have us handle the SSL connection ourselves.
>  
>  I'm acking the patch, contingent upon the regression test suite
>  completing successfully. 
Regression tests all passed. Pushing to master and sssd-1-5.