X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms%2Fsrc%2Forg%2Fargeo%2Fcms%2Fauth%2FCmsAuthUtils.java;h=e9462c3add31cb7dbd0ef16f48afb00e2225a611;hb=3577df2ca5eee54db1ac39a7bd5ed7bdc8824a67;hp=dde2d73f50efffa23f6267d454e9b9c24f75de76;hpb=088c1b517a543e935d8ab65c3b2fd2d0269b551d;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms/src/org/argeo/cms/auth/CmsAuthUtils.java b/org.argeo.cms/src/org/argeo/cms/auth/CmsAuthUtils.java
index dde2d73f5..e9462c3ad 100644
--- a/org.argeo.cms/src/org/argeo/cms/auth/CmsAuthUtils.java
+++ b/org.argeo.cms/src/org/argeo/cms/auth/CmsAuthUtils.java
@@ -1,6 +1,7 @@
 package org.argeo.cms.auth;
 
 import java.security.Principal;
+import java.util.Collection;
 import java.util.Locale;
 import java.util.Set;
 import java.util.UUID;
@@ -12,22 +13,22 @@ import javax.security.auth.x500.X500Principal;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpSession;
 
-//import org.apache.jackrabbit.core.security.AnonymousPrincipal;
-//import org.apache.jackrabbit.core.security.SecurityConstants;
-//import org.apache.jackrabbit.core.security.principal.AdminPrincipal;
-import org.argeo.cms.CmsException;
+import org.argeo.api.NodeConstants;
+import org.argeo.api.security.AnonymousPrincipal;
+import org.argeo.api.security.DataAdminPrincipal;
+import org.argeo.api.security.NodeSecurityUtils;
 import org.argeo.cms.internal.auth.CmsSessionImpl;
 import org.argeo.cms.internal.auth.ImpliedByPrincipal;
 import org.argeo.cms.internal.http.WebCmsSessionImpl;
 import org.argeo.cms.internal.kernel.Activator;
-import org.argeo.node.NodeConstants;
-import org.argeo.node.security.AnonymousPrincipal;
-import org.argeo.node.security.DataAdminPrincipal;
-import org.argeo.node.security.NodeSecurityUtils;
 import org.argeo.osgi.useradmin.AuthenticatingUser;
+import org.osgi.framework.BundleContext;
+import org.osgi.framework.InvalidSyntaxException;
+import org.osgi.framework.ServiceReference;
 import org.osgi.service.http.HttpContext;
 import org.osgi.service.useradmin.Authorization;
 
+/** Centrlaises security related registrations. */
 class CmsAuthUtils {
 	// Standard
 	final static String SHARED_STATE_NAME = AuthenticatingUser.SHARED_STATE_NAME;
@@ -40,9 +41,10 @@ class CmsAuthUtils {
 	final static String SHARED_STATE_SPNEGO_TOKEN = "org.argeo.cms.auth.spnegoToken";
 	final static String SHARED_STATE_SPNEGO_OUT_TOKEN = "org.argeo.cms.auth.spnegoOutToken";
 	final static String SHARED_STATE_CERTIFICATE_CHAIN = "org.argeo.cms.auth.certificateChain";
+	final static String SHARED_STATE_REMOTE_ADDR = "org.argeo.cms.auth.remote.addr";
+	final static String SHARED_STATE_REMOTE_PORT = "org.argeo.cms.auth.remote.port";
 
-	static void addAuthorization(Subject subject, Authorization authorization, Locale locale,
-			HttpServletRequest request) {
+	static void addAuthorization(Subject subject, Authorization authorization) {
 		assert subject != null;
 		checkSubjectEmpty(subject);
 		assert authorization != null;
@@ -70,8 +72,10 @@ class CmsAuthUtils {
 				NodeSecurityUtils.checkUserName(name);
 				userPrincipal = new X500Principal(name.toString());
 				principals.add(userPrincipal);
-				// principals.add(new ImpliedByPrincipal(NodeSecurityUtils.ROLE_USER_NAME,
-				// userPrincipal));
+
+				if (Activator.isSingleUser()) {
+					principals.add(new ImpliedByPrincipal(NodeSecurityUtils.ROLE_ADMIN_NAME, userPrincipal));
+				}
 			}
 
 			// Add roles provided by authorization
@@ -90,10 +94,8 @@ class CmsAuthUtils {
 			}
 
 		} catch (InvalidNameException e) {
-			throw new CmsException("Cannot commit", e);
+			throw new IllegalArgumentException("Cannot commit", e);
 		}
-
-		// registerSessionAuthorization(request, subject, authorization, locale);
 	}
 
 	private static void checkSubjectEmpty(Subject subject) {
@@ -141,7 +143,7 @@ class CmsAuthUtils {
 						cmsSession.close();
 						cmsSession = null;
 					} else if (!authorization.getName().equals(cmsSession.getAuthorization().getName())) {
-						throw new CmsException("Inconsistent user " + authorization.getName()
+						throw new IllegalStateException("Inconsistent user " + authorization.getName()
 								+ " for existing CMS session " + cmsSession);
 					}
 					// keyring
@@ -166,7 +168,7 @@ class CmsAuthUtils {
 					UUID storedSessionId = subject.getPrivateCredentials(CmsSessionId.class).iterator().next()
 							.getUuid();
 					// if (storedSessionId.equals(httpSessionId.getValue()))
-					throw new CmsException(
+					throw new IllegalStateException(
 							"Subject already logged with session " + storedSessionId + " (not " + nodeSessionId + ")");
 				}
 			}
@@ -175,6 +177,29 @@ class CmsAuthUtils {
 		}
 	}
 
+	public static CmsSession cmsSessionFromHttpSession(BundleContext bc, String httpSessionId) {
+		Authorization authorization = null;
+		Collection<ServiceReference<CmsSession>> sr;
+		try {
+			sr = bc.getServiceReferences(CmsSession.class,
+					"(" + CmsSession.SESSION_LOCAL_ID + "=" + httpSessionId + ")");
+		} catch (InvalidSyntaxException e) {
+			throw new IllegalArgumentException("Cannot get CMS session for id " + httpSessionId, e);
+		}
+		CmsSession cmsSession;
+		if (sr.size() == 1) {
+			cmsSession = bc.getService(sr.iterator().next());
+//			locale = cmsSession.getLocale();
+			authorization = cmsSession.getAuthorization();
+			if (authorization.getName() == null)
+				return null;// anonymous is not sufficient
+		} else if (sr.size() == 0)
+			return null;
+		else
+			throw new IllegalStateException(sr.size() + ">1 web sessions detected for http session " + httpSessionId);
+		return cmsSession;
+	}
+
 	public static <T extends Principal> T getSinglePrincipal(Subject subject, Class<T> clss) {
 		Set<T> principals = subject.getPrincipals(clss);
 		if (principals.isEmpty())