[Freeipa-devel] [PATCH] 847 log batch commands, fix logging

Rob Crittenden rcritten at redhat.com
Fri Aug 19 20:36:39 UTC 2011


Rob Crittenden wrote:
> Martin Kosek wrote:
>> On Thu, 2011-08-18 at 16:49 -0400, Rob Crittenden wrote:
>>> Log each call and result for commands in a batch call.
>>>
>>> This also fixes logging in general. I broke it pretty badly when trying
>>> to properly report JSON errors. Both should work now.
>>>
>>> rob
>>
>> ACK. Works fine, tests are clean. I won't push the patch right now in
>> case WebUI guys wouldn't like it.
>>
>> Martin
>>
>
> Endi and Adam acked in IRC. I added the principal to the logging output
> to match the other logs.
>
> Pushed to master and ipa-2-1
>

Endi noticed that I didn't add the principal to successful individual 
batch sub-commands. I added that as a one-liner.

rob




More information about the Freeipa-devel mailing list