[Yanel-commits] rev 45808 - public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet

[michi at wyona.com]

Author: michi
Date: 2009-12-02 16:41:04 +0100 (Wed, 02 Dec 2009)
New Revision: 45808

Modified:
   public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
Log:
log realm and user if exists

Modified: public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
===================================================================
--- public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java	2009-12-02 14:24:23 UTC (rev 45807)
+++ public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java	2009-12-02 15:41:04 UTC (rev 45808)
@@ -2224,7 +2224,6 @@
             //String userID = getEnvironment(request, response).getIdentity().getUsername();
             Identity identity = getIdentity(request, map);
             if (identity != null && identity.getUsername() != null) {
-                log.warn("DEBUG: Log browser history of user '"+identity.getUsername()+"'.");
                 Realm realm = map.getRealm(request.getServletPath());
                 User user = realm.getIdentityManager().getUserManager().getUser(identity.getUsername());
                 // The log should be attached to the user, because realms can share a UserManager, but the UserManager API has no mean to save such data, so how should we do this?
@@ -2244,6 +2243,10 @@
                  - Analyze mime type (advantage: no additional code/requests necessary)
                  - Log analysis (no special tracking required)
 */
+                String requestURL = request.getRequestURL().toString();
+                if (requestURL.endsWith("html")) { // TODO: Check the mime-type instead the suffix or use JavaScript or Pixel
+                    logAccess.info(requestURL + " " + realm.getID() + " " + identity.getUsername());
+                }
             } else {
                 // NOTE: Log access of anonymous user
                 String requestURL = request.getRequestURL().toString();