You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by de...@apache.org on 2010/05/25 03:52:39 UTC

svn commit: r947892 - /geronimo/server/branches/2.2/plugins/tomcat/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/security/SecurityValve.java

Author: delos
Date: Tue May 25 01:52:39 2010
New Revision: 947892

URL: http://svn.apache.org/viewvc?rev=947892&view=rev
Log:
GERONIMO-5010 Instead of response with HTTP 403, just return 302 to redirect to secure port

Modified:
    geronimo/server/branches/2.2/plugins/tomcat/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/security/SecurityValve.java

Modified: geronimo/server/branches/2.2/plugins/tomcat/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/security/SecurityValve.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/tomcat/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/security/SecurityValve.java?rev=947892&r1=947891&r2=947892&view=diff
==============================================================================
--- geronimo/server/branches/2.2/plugins/tomcat/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/security/SecurityValve.java (original)
+++ geronimo/server/branches/2.2/plugins/tomcat/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/security/SecurityValve.java Tue May 25 01:52:39 2010
@@ -51,7 +51,7 @@ public class SecurityValve extends Valve
         if (!authorizer.hasUserDataPermissions(request, constraints)) {
             //TODO redirect to secure port?
             if (!response.isError()) {
-                response.sendError(Response.SC_FORBIDDEN);
+                response.sendError(Response.SC_MOVED_TEMPORARILY);
             }
             return;
         }