[Freeipa-devel] [PATCH 0051-0052] Log successful reconnection to LDAP server

[Petr Spacek]

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