You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by jo...@apache.org on 2006/10/27 22:02:35 UTC

svn commit: r468513 - /maven/archiva/trunk/archiva-webapp/src/main/java/org/apache/maven/archiva/web/servlet/repository/RepositoryAccess.java

Author: joakime
Date: Fri Oct 27 13:02:35 2006
New Revision: 468513

URL: http://svn.apache.org/viewvc?view=rev&rev=468513
Log:
* Adding HEAD as ACCESS only method.
* Removing full stacktrace on authz denied logging.
* Adding remote ip address on authz denied.


Modified:
    maven/archiva/trunk/archiva-webapp/src/main/java/org/apache/maven/archiva/web/servlet/repository/RepositoryAccess.java

Modified: maven/archiva/trunk/archiva-webapp/src/main/java/org/apache/maven/archiva/web/servlet/repository/RepositoryAccess.java
URL: http://svn.apache.org/viewvc/maven/archiva/trunk/archiva-webapp/src/main/java/org/apache/maven/archiva/web/servlet/repository/RepositoryAccess.java?view=diff&rev=468513&r1=468512&r2=468513
==============================================================================
--- maven/archiva/trunk/archiva-webapp/src/main/java/org/apache/maven/archiva/web/servlet/repository/RepositoryAccess.java (original)
+++ maven/archiva/trunk/archiva-webapp/src/main/java/org/apache/maven/archiva/web/servlet/repository/RepositoryAccess.java Fri Oct 27 13:02:35 2006
@@ -81,6 +81,7 @@
     static
     {
         ACCESS_METHODS = new ArrayList();
+        ACCESS_METHODS.add( "HEAD" );
         ACCESS_METHODS.add( "GET" );
         ACCESS_METHODS.add( "PROPFIND" );
         ACCESS_METHODS.add( "OPTIONS" );
@@ -185,7 +186,9 @@
             {
                 if ( authzResult.getException() != null )
                 {
-                    getLogger().warn( "Authorization Denied", authzResult.getException() );
+                    getLogger().warn( "Authorization Denied [ip=" + request.getRemoteAddr() + ",isWriteRequest="
+                                          + isWriteRequest + ",permission=" + permission + "] : "
+                                          + authzResult.getException().getMessage() );
                 }
                 
                 // Issue HTTP Challenge.