You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by fm...@apache.org on 2011/11/09 07:35:31 UTC

svn commit: r1199628 - /sling/trunk/bundles/jcr/davex/src/main/java/org/apache/sling/jcr/davex/impl/servlets/AuthHttpContext.java

Author: fmeschbe
Date: Wed Nov  9 06:35:31 2011
New Revision: 1199628

URL: http://svn.apache.org/viewvc?rev=1199628&view=rev
Log:
SLING-2167 Reverting my last change thus not forcing HTTP Basic. The real solution is to make sure authentication on the davex path is not enforced by Sling authentication.

Modified:
    sling/trunk/bundles/jcr/davex/src/main/java/org/apache/sling/jcr/davex/impl/servlets/AuthHttpContext.java

Modified: sling/trunk/bundles/jcr/davex/src/main/java/org/apache/sling/jcr/davex/impl/servlets/AuthHttpContext.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/jcr/davex/src/main/java/org/apache/sling/jcr/davex/impl/servlets/AuthHttpContext.java?rev=1199628&r1=1199627&r2=1199628&view=diff
==============================================================================
--- sling/trunk/bundles/jcr/davex/src/main/java/org/apache/sling/jcr/davex/impl/servlets/AuthHttpContext.java (original)
+++ sling/trunk/bundles/jcr/davex/src/main/java/org/apache/sling/jcr/davex/impl/servlets/AuthHttpContext.java Wed Nov  9 06:35:31 2011
@@ -23,7 +23,6 @@ import javax.servlet.http.HttpServletReq
 import javax.servlet.http.HttpServletResponse;
 
 import org.apache.sling.auth.core.AuthenticationSupport;
-import org.apache.sling.auth.core.spi.AuthenticationHandler;
 import org.osgi.service.http.HttpContext;
 
 class AuthHttpContext implements HttpContext {
@@ -68,7 +67,6 @@ class AuthHttpContext implements HttpCon
     throws IOException {
         final AuthenticationSupport localAuthenticator = this.authenticator;
         if ( localAuthenticator != null ) {
-            request.setAttribute(AuthenticationHandler.REQUEST_LOGIN_PARAMETER, "BASIC");
             return localAuthenticator.handleSecurity(request, response);
         }
         // send 503/SERVICE UNAVAILABLE, flush to ensure delivery