X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms%2Fsrc%2Forg%2Fargeo%2Fcms%2Finternal%2Fhttp%2FCmsSessionProvider.java;h=a1ddcb0babb26315ac569bf0297e03f7094e6a43;hb=a07d415f007b9eb33b68f7c266e729105d214add;hp=37ba5cdb15332a440e1dac222ecaf7ed16d1db28;hpb=34ba1b915e1d406f6574c0be93e1e9da3eab1978;p=lgpl%2Fargeo-commons.git diff --git a/org.argeo.cms/src/org/argeo/cms/internal/http/CmsSessionProvider.java b/org.argeo.cms/src/org/argeo/cms/internal/http/CmsSessionProvider.java index 37ba5cdb1..a1ddcb0ba 100644 --- a/org.argeo.cms/src/org/argeo/cms/internal/http/CmsSessionProvider.java +++ b/org.argeo.cms/src/org/argeo/cms/internal/http/CmsSessionProvider.java @@ -12,21 +12,21 @@ import javax.servlet.http.HttpServletRequest; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.apache.jackrabbit.server.SessionProvider; -import org.argeo.cms.auth.CmsSession; +import org.argeo.cms.internal.auth.CmsSessionImpl; import org.argeo.jcr.JcrUtils; /** * Implements an open session in view patter: a new JCR session is created for * each request */ -class CmsSessionProvider implements SessionProvider, Serializable { +public class CmsSessionProvider implements SessionProvider, Serializable { private static final long serialVersionUID = -1358136599534938466L; private final static Log log = LogFactory.getLog(CmsSessionProvider.class); private final String alias; - private LinkedHashMap cmsSessions = new LinkedHashMap<>(); + private LinkedHashMap cmsSessions = new LinkedHashMap<>(); public CmsSessionProvider(String alias) { this.alias = alias; @@ -35,43 +35,46 @@ class CmsSessionProvider implements SessionProvider, Serializable { public Session getSession(HttpServletRequest request, Repository rep, String workspace) throws javax.jcr.LoginException, ServletException, RepositoryException { - CmsSession cmsSession = WebCmsSessionImpl.getCmsSession(request); -// if (cmsSession == null) -// return anonymousSession(request, rep, workspace); + CmsSessionImpl cmsSession = WebCmsSessionImpl.getCmsSession(request); + // if (cmsSession == null) + // return anonymousSession(request, rep, workspace); if (log.isTraceEnabled()) { - log.debug("Get JCR session from " + cmsSession); + log.trace("Get JCR session from " + cmsSession); } Session session = cmsSession.getDataSession(alias, workspace, rep); cmsSessions.put(session, cmsSession); return session; } -// private synchronized Session anonymousSession(HttpServletRequest request, Repository repository, String workspace) { -// // TODO rather log in here as anonymous? -// LoginContext lc = (LoginContext) request.getAttribute(NodeConstants.LOGIN_CONTEXT_ANONYMOUS); -// if (lc == null) -// throw new CmsException("No login context available"); -// // optimize -// Session session; -// try { -// session = Subject.doAs(lc.getSubject(), new PrivilegedExceptionAction() { -// @Override -// public Session run() throws Exception { -// return repository.login(workspace); -// } -// }); -// } catch (Exception e) { -// throw new CmsException("Cannot log in to JCR", e); -// } -// return session; -// } + // private synchronized Session anonymousSession(HttpServletRequest request, + // Repository repository, String workspace) { + // // TODO rather log in here as anonymous? + // LoginContext lc = (LoginContext) + // request.getAttribute(NodeConstants.LOGIN_CONTEXT_ANONYMOUS); + // if (lc == null) + // throw new CmsException("No login context available"); + // // optimize + // Session session; + // try { + // session = Subject.doAs(lc.getSubject(), new + // PrivilegedExceptionAction() { + // @Override + // public Session run() throws Exception { + // return repository.login(workspace); + // } + // }); + // } catch (Exception e) { + // throw new CmsException("Cannot log in to JCR", e); + // } + // return session; + // } public synchronized void releaseSession(Session session) { if (cmsSessions.containsKey(session)) { - CmsSession cmsSession = cmsSessions.get(session); + CmsSessionImpl cmsSession = cmsSessions.get(session); cmsSession.releaseDataSession(alias, session); } else { - log.warn("JCR session "+session+" not found in CMS session list. Logging it out..."); + log.warn("JCR session " + session + " not found in CMS session list. Logging it out..."); JcrUtils.logoutQuietly(session); } }