Mercurial > hg > digilib

changeset 1494:589f47478095

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
cleanups.
author robcast
date Wed, 30 Mar 2016 11:21:54 +0200
parents 14a28f8bcc7d
children bc66091ba443
files servlet/src/main/java/digilib/auth/PathAuthzOps.java servlet3/src/main/java/digilib/servlet/Scaler.java
diffstat 2 files changed, 2 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/servlet/src/main/java/digilib/auth/PathAuthzOps.java	Wed Mar 30 10:28:08 2016 +0200
+++ b/servlet/src/main/java/digilib/auth/PathAuthzOps.java	Wed Mar 30 11:21:54 2016 +0200
@@ -30,8 +30,6 @@
 import java.util.List;
 import java.util.Map;
 
-import javax.servlet.http.HttpServletRequest;
-
 import digilib.conf.DigilibConfiguration;
 import digilib.conf.DigilibServletConfiguration;
 import digilib.conf.DigilibServletRequest;
@@ -105,9 +103,7 @@
      */
     public List<String> rolesForPath(DigilibServletRequest dlRequest) throws digilib.auth.AuthOpException {
         String filepath = dlRequest.getFilePath();
-        HttpServletRequest request = dlRequest.getServletRequest();
-        logger.debug("rolesForPath (" + filepath + ") by [" + request.getRemoteAddr() + "]");
-
+        logger.debug("rolesForPath: " + filepath);
         // which roles are required?
         List<String> required = authPaths.match(filepath);
         return required;
--- a/servlet3/src/main/java/digilib/servlet/Scaler.java	Wed Mar 30 10:28:08 2016 +0200
+++ b/servlet3/src/main/java/digilib/servlet/Scaler.java	Wed Mar 30 11:21:54 2016 +0200
@@ -277,7 +277,7 @@
                 // is the current request/user authorized?
                 if (!authzOp.isAuthorized(dlRequest)) {
                     // send deny answer and abort
-                    throw new AuthOpException();
+                    throw new AuthOpException("Access denied!");
                 }
             }