]> git.argeo.org Git - lgpl/argeo-commons.git/blobdiff - org.argeo.cms/src/org/argeo/cms/internal/kernel/NodeHttp.java
Simplify authentication
[lgpl/argeo-commons.git] / org.argeo.cms / src / org / argeo / cms / internal / kernel / NodeHttp.java
index f3a90ebfdb76b60a57b3df2613ae1332b125f423..5e1f39acb1f7a57d0651a0976a5c0cc786a781a1 100644 (file)
 package org.argeo.cms.internal.kernel;
 
 import java.io.IOException;
+import java.security.cert.X509Certificate;
 import java.util.Enumeration;
-import java.util.Properties;
-import java.util.StringTokenizer;
 
 import javax.servlet.FilterChain;
-import javax.servlet.Servlet;
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import javax.servlet.http.HttpSession;
 
-import org.apache.commons.codec.binary.Base64;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.argeo.cms.CmsException;
-import org.argeo.jackrabbit.servlet.OpenInViewSessionProvider;
-import org.argeo.jackrabbit.servlet.RemotingServlet;
-import org.argeo.jackrabbit.servlet.WebdavServlet;
 import org.argeo.jcr.ArgeoJcrConstants;
 import org.eclipse.equinox.http.servlet.ExtendedHttpService;
-import org.osgi.framework.BundleContext;
-import org.osgi.service.http.NamespaceException;
-import org.osgi.util.tracker.ServiceTracker;
-import org.springframework.security.authentication.AuthenticationManager;
-import org.springframework.security.authentication.UsernamePasswordAuthenticationToken;
-import org.springframework.security.core.Authentication;
-import org.springframework.security.core.context.SecurityContext;
-import org.springframework.security.core.context.SecurityContextHolder;
 
 /**
  * Intercepts and enriches http access, mainly focusing on security and
  * transactionality.
  */
+@Deprecated
 class NodeHttp implements KernelConstants, ArgeoJcrConstants {
        private final static Log log = LogFactory.getLog(NodeHttp.class);
 
-       private final static String ATTR_AUTH = "auth";
-       private final static String HEADER_AUTHORIZATION = "Authorization";
-       private final static String HEADER_WWW_AUTHENTICATE = "WWW-Authenticate";
-
-       private final AuthenticationManager authenticationManager;
-       private final BundleContext bundleContext;
-       private ExtendedHttpService httpService;
-
-       // FIXME Make it more unique
-       private String httpAuthRealm = "Argeo";
-
        // Filters
        private final RootFilter rootFilter;
+
        // private final DoSFilter dosFilter;
        // private final QoSFilter qosFilter;
 
-       // remoting
-       private OpenInViewSessionProvider sessionProvider;
-       private WebdavServlet publicWebdavServlet;
-       private WebdavServlet privateWebdavServlet;
-       private RemotingServlet publicRemotingServlet;
-       private RemotingServlet privateRemotingServlet;
-
-       NodeHttp(BundleContext bundleContext, JackrabbitNode node,
-                       NodeSecurity authenticationManager) {
-               this.bundleContext = bundleContext;
-               this.authenticationManager = authenticationManager;
-
-               // Equinox dependency
-               ServiceTracker<ExtendedHttpService, ExtendedHttpService> st = new ServiceTracker<ExtendedHttpService, ExtendedHttpService>(
-                               bundleContext, ExtendedHttpService.class, null);
-               st.open();
-               try {
-                       httpService = st.waitForService(1000);
-               } catch (InterruptedException e) {
-                       httpService = null;
-               }
-
-               if (httpService == null)
-                       throw new CmsException("Could not find "
-                                       + ExtendedHttpService.class + " service.");
-
-               // Filters
+       NodeHttp(ExtendedHttpService httpService) {
                rootFilter = new RootFilter();
                // dosFilter = new CustomDosFilter();
                // qosFilter = new QoSFilter();
 
-               // DAV
-               sessionProvider = new OpenInViewSessionProvider();
-               publicWebdavServlet = new WebdavServlet(node, sessionProvider);
-               privateWebdavServlet = new WebdavServlet(node, sessionProvider);
-               publicRemotingServlet = new RemotingServlet(node, sessionProvider);
-               privateRemotingServlet = new RemotingServlet(node, sessionProvider);
-       }
-
-       void publish() {
                try {
-                       registerWebdavServlet(PATH_WEBDAV_PUBLIC, ALIAS_NODE, true,
-                                       publicWebdavServlet);
-                       registerWebdavServlet(PATH_WEBDAV_PRIVATE, ALIAS_NODE, false,
-                                       privateWebdavServlet);
-                       registerRemotingServlet(PATH_REMOTING_PUBLIC, ALIAS_NODE, true,
-                                       publicRemotingServlet);
-                       registerRemotingServlet(PATH_REMOTING_PRIVATE, ALIAS_NODE, false,
-                                       privateRemotingServlet);
-
-                       // httpService.registerFilter("/", dosFilter, null, null);
                        httpService.registerFilter("/", rootFilter, null, null);
-                       // httpService.registerFilter("/", qosFilter, null, null);
                } catch (Exception e) {
-                       throw new CmsException("Cannot publish HTTP services to OSGi", e);
+                       throw new CmsException("Cannot register filters", e);
                }
        }
 
-       private void registerWebdavServlet(String pathPrefix, String alias,
-                       Boolean anonymous, WebdavServlet webdavServlet)
-                       throws NamespaceException, ServletException {
-               String path = pathPrefix + "/" + alias;
-               Properties initParameters = new Properties();
-               initParameters.setProperty(WebdavServlet.INIT_PARAM_RESOURCE_CONFIG,
-                               KernelConstants.WEBDAV_CONFIG);
-               initParameters.setProperty(
-                               WebdavServlet.INIT_PARAM_RESOURCE_PATH_PREFIX, path);
-               httpService.registerFilter(path, anonymous ? new AnonymousFilter()
-                               : new DavFilter(), null, null);
-               // Cast to servlet because of a weird behaviour in Eclipse
-               httpService.registerServlet(path, (Servlet) webdavServlet,
-                               initParameters, null);
-       }
-
-       private void registerRemotingServlet(String pathPrefix, String alias,
-                       Boolean anonymous, RemotingServlet remotingServlet)
-                       throws NamespaceException, ServletException {
-               String path = pathPrefix + "/" + alias;
-               Properties initParameters = new Properties();
-               initParameters.setProperty(
-                               RemotingServlet.INIT_PARAM_RESOURCE_PATH_PREFIX, path);
-
-               // Looks like a bug in Jackrabbit remoting init
-               initParameters.setProperty(RemotingServlet.INIT_PARAM_HOME,
-                               KernelUtils.getOsgiInstanceDir(bundleContext)
-                                               + "/tmp/jackrabbit");
-               initParameters.setProperty(RemotingServlet.INIT_PARAM_TMP_DIRECTORY,
-                               "remoting");
-               // Cast to servlet because of a weird behaviour in Eclipse
-               httpService.registerFilter(path, anonymous ? new AnonymousFilter()
-                               : new DavFilter(), null, null);
-               httpService.registerServlet(path, (Servlet) remotingServlet,
-                               initParameters, null);
-       }
-
-       private Boolean isSessionAuthenticated(HttpSession httpSession) {
-               SecurityContext contextFromSession = (SecurityContext) httpSession
-                               .getAttribute(SPRING_SECURITY_CONTEXT_KEY);
-               return contextFromSession != null;
-       }
-
-       private void requestBasicAuth(HttpSession httpSession,
-                       HttpServletResponse response) {
-               response.setStatus(401);
-               response.setHeader(HEADER_WWW_AUTHENTICATE, "basic realm=\""
-                               + httpAuthRealm + "\"");
-               httpSession.setAttribute(ATTR_AUTH, Boolean.TRUE);
-       }
-
-       private UsernamePasswordAuthenticationToken basicAuth(String authHeader) {
-               if (authHeader != null) {
-                       StringTokenizer st = new StringTokenizer(authHeader);
-                       if (st.hasMoreTokens()) {
-                               String basic = st.nextToken();
-                               if (basic.equalsIgnoreCase("Basic")) {
-                                       try {
-                                               String credentials = new String(Base64.decodeBase64(st
-                                                               .nextToken()), "UTF-8");
-                                               // log.debug("Credentials: " + credentials);
-                                               int p = credentials.indexOf(":");
-                                               if (p != -1) {
-                                                       String login = credentials.substring(0, p).trim();
-                                                       String password = credentials.substring(p + 1)
-                                                                       .trim();
-
-                                                       return new UsernamePasswordAuthenticationToken(
-                                                                       login, password.toCharArray());
-                                               } else {
-                                                       throw new CmsException(
-                                                                       "Invalid authentication token");
-                                               }
-                                       } catch (Exception e) {
-                                               throw new CmsException(
-                                                               "Couldn't retrieve authentication", e);
-                                       }
-                               }
-                       }
-               }
-               throw new CmsException("Couldn't retrieve authentication");
+       public void destroy() {
        }
 
        /** Intercepts all requests. Authenticates. */
@@ -202,37 +53,42 @@ class NodeHttp implements KernelConstants, ArgeoJcrConstants {
                                HttpServletRequest request, HttpServletResponse response,
                                FilterChain filterChain) throws IOException, ServletException {
                        if (log.isTraceEnabled()) {
-                               log.debug(request.getContextPath());
-                               log.debug(request.getServletPath());
-                               log.debug(request.getRequestURI());
-                               log.debug(request.getQueryString());
-                               StringBuilder buf = new StringBuilder();
-                               Enumeration<String> en = request.getHeaderNames();
-                               while (en.hasMoreElements()) {
-                                       String header = en.nextElement();
-                                       Enumeration<String> values = request.getHeaders(header);
-                                       while (values.hasMoreElements())
-                                               buf.append("  " + header + ": " + values.nextElement());
-                                       buf.append('\n');
-                               }
-                               log.debug("\n" + buf);
+                               log.trace(request.getRequestURL().append(
+                                               request.getQueryString() != null ? "?"
+                                                               + request.getQueryString() : ""));
+                               logRequest(request);
                        }
 
                        String servletPath = request.getServletPath();
 
+                       // client certificate
+                       X509Certificate clientCert = extractCertificate(request);
+                       if (clientCert != null) {
+                               // TODO authenticate
+                               // if (log.isDebugEnabled())
+                               // log.debug(clientCert.getSubjectX500Principal().getName());
+                       }
+
                        // skip data
                        if (servletPath.startsWith(PATH_DATA)) {
                                filterChain.doFilter(request, response);
                                return;
                        }
 
+                       // skip /ui (workbench) for the time being
+                       if (servletPath.startsWith(PATH_WORKBENCH)) {
+                               filterChain.doFilter(request, response);
+                               return;
+                       }
+
                        // redirect long RWT paths to anchor
                        String path = request.getRequestURI().substring(
                                        servletPath.length());
                        int pathLength = path.length();
                        if (pathLength != 0 && (path.charAt(0) == '/')
                                        && !servletPath.endsWith("rwt-resources")
-                                       && !path.equals("/")) {
+                                       && !path.startsWith(KernelConstants.PATH_WORKBENCH)
+                                       && path.lastIndexOf('/') != 0) {
                                String newLocation = request.getServletPath() + "#" + path;
                                response.setHeader("Location", newLocation);
                                response.setStatus(HttpServletResponse.SC_FOUND);
@@ -244,53 +100,40 @@ class NodeHttp implements KernelConstants, ArgeoJcrConstants {
                }
        }
 
-       /** Intercepts all requests. Authenticates. */
-       class AnonymousFilter extends HttpFilter {
-               @Override
-               public void doFilter(HttpSession httpSession,
-                               HttpServletRequest request, HttpServletResponse response,
-                               FilterChain filterChain) throws IOException, ServletException {
-
-                       // Authenticate from session
-                       if (isSessionAuthenticated(httpSession)) {
-                               filterChain.doFilter(request, response);
-                               return;
-                       }
+       private void logRequest(HttpServletRequest request) {
+               log.debug("contextPath=" + request.getContextPath());
+               log.debug("servletPath=" + request.getServletPath());
+               log.debug("requestURI=" + request.getRequestURI());
+               log.debug("queryString=" + request.getQueryString());
+               StringBuilder buf = new StringBuilder();
+               // headers
+               Enumeration<String> en = request.getHeaderNames();
+               while (en.hasMoreElements()) {
+                       String header = en.nextElement();
+                       Enumeration<String> values = request.getHeaders(header);
+                       while (values.hasMoreElements())
+                               buf.append("  " + header + ": " + values.nextElement());
+                       buf.append('\n');
+               }
 
-                       KernelUtils.anonymousLogin(authenticationManager);
-                       filterChain.doFilter(request, response);
+               // attributed
+               Enumeration<String> an = request.getAttributeNames();
+               while (an.hasMoreElements()) {
+                       String attr = an.nextElement();
+                       Object value = request.getAttribute(attr);
+                       buf.append("  " + attr + ": " + value);
+                       buf.append('\n');
                }
+               log.debug("\n" + buf);
        }
 
-       /** Intercepts all requests. Authenticates. */
-       class DavFilter extends HttpFilter {
-
-               @Override
-               public void doFilter(HttpSession httpSession,
-                               HttpServletRequest request, HttpServletResponse response,
-                               FilterChain filterChain) throws IOException, ServletException {
-
-                       // Authenticate from session
-                       // if (isSessionAuthenticated(httpSession)) {
-                       // filterChain.doFilter(request, response);
-                       // return;
-                       // }
-
-                       // Process basic auth
-                       String basicAuth = request.getHeader(HEADER_AUTHORIZATION);
-                       if (basicAuth != null) {
-                               UsernamePasswordAuthenticationToken token = basicAuth(basicAuth);
-                               Authentication auth = authenticationManager.authenticate(token);
-                               SecurityContextHolder.getContext().setAuthentication(auth);
-                               httpSession.setAttribute(SPRING_SECURITY_CONTEXT_KEY,
-                                               SecurityContextHolder.getContext());
-                               httpSession.setAttribute(ATTR_AUTH, Boolean.FALSE);
-                               filterChain.doFilter(request, response);
-                               return;
-                       }
-
-                       requestBasicAuth(httpSession, response);
+       private X509Certificate extractCertificate(HttpServletRequest req) {
+               X509Certificate[] certs = (X509Certificate[]) req
+                               .getAttribute("javax.servlet.request.X509Certificate");
+               if (null != certs && certs.length > 0) {
+                       return certs[0];
                }
+               return null;
        }
 
        // class CustomDosFilter extends DoSFilter {