logging anonymous requests

caelum · Jul 31, 2014 · 4db42bc · 4db42bc
1 parent bafbe38
commit 4db42bc
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/src/main/java/org/mamute/providers/DefaultViewObjects.java b/src/main/java/org/mamute/providers/DefaultViewObjects.java
@@ -69,7 +69,6 @@ private String getCurrentUrl() {
 			url = "http://" + host + req.getRequestURI(); 
 		}
 		if(url.endsWith("/")) url = url.split(SLASH_AT_END)[0];
-		LOG.debug("setting url: " + url);
 		return url;
 	}
 

diff --git a/src/main/java/org/mamute/util/UserLogger.java b/src/main/java/org/mamute/util/UserLogger.java
@@ -21,16 +21,19 @@ public class UserLogger {
 	private static Logger LOG = Logger.getLogger(UserLogger.class);
 
 	public void logUser(@Observes MethodReady methodReady) {
+		String userName = "anonymous";
 		if (loggedUser.isLoggedIn()) {
-			String realIp = request.getHeader("X-Real-IP");
 			User user = loggedUser.getCurrent();
 			String name = user.getName();
 			String id = user.getId().toString();
-			String uri = request.getRequestURI();
-			String method = request.getMethod();
-			String log = format("%s -> %s from %s (id=%s) %s", method, uri, name, id,  realIp);
-			LOG.info(log);
+			userName = format("%s (id=%s)", name, id);
 		}
 
+		String realIp = request.getHeader("X-Real-IP");
+		String uri = request.getRequestURI();
+		String method = request.getMethod();
+		String log = format("%s -> %s from %s %s", method, uri, userName,  realIp);
+		LOG.info(log);
+
 	}
 }