azkaban-aplcache

Removing sessionId from logger statements (#1180) Fixing

6/7/2017 11:57:18 PM

Details

diff --git a/azkaban-web-server/src/restli/java/azkaban/restli/ProjectManagerResource.java b/azkaban-web-server/src/restli/java/azkaban/restli/ProjectManagerResource.java
index 4112f81..2d6fa62 100644
--- a/azkaban-web-server/src/restli/java/azkaban/restli/ProjectManagerResource.java
+++ b/azkaban-web-server/src/restli/java/azkaban/restli/ProjectManagerResource.java
@@ -56,8 +56,7 @@ public class ProjectManagerResource extends ResourceContextHolder {
       @ActionParam("packageUrl") final String packageUrl)
       throws ProjectManagerException, RestLiServiceException, UserManagerException,
       ServletException, IOException {
-    logger.info("Deploy called. {sessionId: " + sessionId + ", projectName: "
-        + projectName + ", packageUrl:" + packageUrl + "}");
+    logger.info("Deploy called. {projectName: " + projectName + ", packageUrl:" + packageUrl + "}");
 
     final String ip = ResourceUtils.getRealClientIpAddr(this.getContext());
     final User user = ResourceUtils.getUserFromSessionId(sessionId, ip);
diff --git a/azkaban-web-server/src/restli/java/azkaban/restli/UserManagerResource.java b/azkaban-web-server/src/restli/java/azkaban/restli/UserManagerResource.java
index 566a28a..0bbed0f 100644
--- a/azkaban-web-server/src/restli/java/azkaban/restli/UserManagerResource.java
+++ b/azkaban-web-server/src/restli/java/azkaban/restli/UserManagerResource.java
@@ -49,8 +49,7 @@ public class UserManagerResource extends ResourceContextHolder {
 
     final Session session = createSession(username, password, ip);
 
-    logger.info("Session id " + session.getSessionId() + " created for user '"
-        + username + "' and ip " + ip);
+    logger.info("Session id created for user '" + username + "' and ip " + ip);
     return session.getSessionId();
   }