We currently log just the connection open/close and the command being executed. This adds something about what the result of the request was, either an exception or the number of records returned.

rob
>From f3cba999cf048a53205c4b226bd5e1959eb9ac8a Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcrit...@redhat.com>
Date: Mon, 29 Mar 2010 16:46:03 -0400
Subject: [PATCH] Log some information on the result of a request

---
 ipaserver/rpcserver.py |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/ipaserver/rpcserver.py b/ipaserver/rpcserver.py
index 967ee33..667a7c4 100644
--- a/ipaserver/rpcserver.py
+++ b/ipaserver/rpcserver.py
@@ -293,8 +293,10 @@ class xmlserver(WSGIExecutioner):
 
     def marshal(self, result, error, _id=None):
         if error:
+            self.info('response: %s: %s', error.__class__.__name__, str(error))
             response = Fault(error.errno, error.strerror)
         else:
+            self.info('response: entries returned %d', result.get('count', 1))
             response = (result,)
         return xml_dumps(response, methodresponse=True)
 
-- 
1.6.2.5

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

Reply via email to