import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.argeo.jcr.ThreadBoundJcrSessionFactory;
+import org.springframework.beans.factory.DisposableBean;
+import org.springframework.beans.factory.FactoryBean;
+import org.springframework.beans.factory.InitializingBean;
import org.springframework.security.Authentication;
import org.springframework.security.context.SecurityContextHolder;
+import org.springframework.security.userdetails.UserDetails;
-public class SecureThreadBoundSession extends ThreadBoundJcrSessionFactory {
+/**
+ * Thread bounded JCR session factory which checks authentication and is
+ * autoconfigured in Spring.
+ */
+public class SecureThreadBoundSession extends ThreadBoundJcrSessionFactory
+ implements FactoryBean, InitializingBean, DisposableBean {
private final static Log log = LogFactory
.getLog(SecureThreadBoundSession.class);
+ public void afterPropertiesSet() throws Exception {
+ init();
+ }
+
+ public void destroy() throws Exception {
+ dispose();
+ }
+
@Override
protected Session preCall(Session session) {
Authentication authentication = SecurityContextHolder.getContext()
.getAuthentication();
if (authentication != null) {
- if (!session.getUserID().equals(
- authentication.getPrincipal().toString())) {
- log.warn("Current session has user ID " + session.getUserID()
- + " while authentication is " + authentication
- + ". Re-login.");
- return login();
+ String userID = session.getUserID();
+ UserDetails userDetails = (UserDetails) authentication.getDetails();
+ if (userDetails != null) {
+ String currentUserName = userDetails.getUsername();
+ if (!userID.equals(currentUserName)) {
+ log.warn("Current session has user ID " + userID
+ + " while logged is user is " + currentUserName
+ + "(authentication=" + authentication + ")"
+ + ". Re-login.");
+ return login();
+ }
}
}
return super.preCall(session);