You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by jz...@apache.org on 2013/01/25 04:02:30 UTC

[2/3] git commit: CLOUDSTACK-505: Added string cleaning to the start and end debug log statements within the API servlet.

CLOUDSTACK-505: Added string cleaning to the start and end debug
log statements within the API servlet.

Conflicts:
	server/src/com/cloud/api/ApiServlet.java


Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/a81d227f
Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/a81d227f
Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/a81d227f

Branch: refs/heads/4.0
Commit: a81d227f1f04d23a17c2ea0a2af2c8f8e476e794
Parents: d5c9ad0
Author: Chip Childers <ch...@gmail.com>
Authored: Mon Dec 17 15:13:59 2012 -0500
Committer: Joe Brockmeier <jz...@zonker.net>
Committed: Thu Jan 24 19:21:58 2013 -0600

----------------------------------------------------------------------
 server/src/com/cloud/api/ApiServlet.java |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/a81d227f/server/src/com/cloud/api/ApiServlet.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/api/ApiServlet.java b/server/src/com/cloud/api/ApiServlet.java
index 8a1d4de..abf2941 100755
--- a/server/src/com/cloud/api/ApiServlet.java
+++ b/server/src/com/cloud/api/ApiServlet.java
@@ -36,6 +36,7 @@ import com.cloud.server.ManagementServer;
 import com.cloud.user.Account;
 import com.cloud.user.AccountService;
 import com.cloud.user.UserContext;
+import com.cloud.utils.StringUtils;
 import com.cloud.utils.component.ComponentLocator;
 import com.cloud.utils.exception.CloudRuntimeException;
 
@@ -122,6 +123,13 @@ public class ApiServlet extends HttpServlet {
         //
         utf8Fixup(req, params);
 
+        // logging the request start and end in management log for easy debugging
+        String reqStr = "";
+        if (s_logger.isDebugEnabled()) {
+            reqStr = auditTrailSb.toString() + " " + req.getQueryString();
+            s_logger.debug("===START=== " + StringUtils.cleanString(reqStr));
+        }
+        
         try {
             HttpSession session = req.getSession(false);
             Object[] responseTypeParam = params.get("response");
@@ -335,6 +343,11 @@ public class ApiServlet extends HttpServlet {
             }
         } finally {
             s_accessLogger.info(auditTrailSb.toString());
+
+            if (s_logger.isDebugEnabled()) {
+                s_logger.debug("===END=== " + StringUtils.cleanString(reqStr));
+            }
+            
             // cleanup user context to prevent from being peeked in other request context
             UserContext.unregisterContext();
         }