On 09/05/2012 04:37 PM, Adam Tkac wrote:
On Wed, Sep 05, 2012 at 03:53:36PM +0200, Petr Spacek wrote:
On 09/05/2012 01:29 PM, Adam Tkac wrote:
On Wed, Aug 15, 2012 at 01:20:08PM +0200, Petr Spacek wrote:
Hello,

this two patches solves upstream ticket
https://fedorahosted.org/bind-dyndb-ldap/ticket/71
"Log successful reconnect"

Patch 51:
     Adds log_info(): logging facility with log level INFO.

Ack.

Pushed to master:
https://fedorahosted.org/bind-dyndb-ldap/changeset/3d5310d29f709f83fabb33f54356efa2acf5582d


Patch 52:
     Logs successful reconnection to LDAP server.

     LDAP connection error handling was modified:
     Errors are handled exclusively by handle_connection_error() now.

     Direct calls to ldap_connect() and ldap_reconnect() should be avoided.

Nack, please check my comments below.

Thanks for your comments! Corrected patches are attached + I replied in-line.

Ack

Pushed to master:
https://fedorahosted.org/bind-dyndb-ldap/changeset/ea9c3287f9e01dd932116e39de6cb592862f7591
(CLEANUP_WITH macro)

and

https://fedorahosted.org/bind-dyndb-ldap/e89735d969dc79d632f2c7776f5e88b934ccce57
(patch itself)

Petr^2 Spacek

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to