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.argeo.jcr.spring.ThreadBoundSession;
import org.springframework.security.Authentication;
import org.springframework.security.context.SecurityContextHolder;
-import org.springframework.security.userdetails.UserDetails;
/**
* Thread bounded JCR session factory which checks authentication and is
* autoconfigured in Spring.
*/
-public class SecureThreadBoundSession extends ThreadBoundJcrSessionFactory
- implements FactoryBean, InitializingBean, DisposableBean {
+public class SecureThreadBoundSession extends ThreadBoundSession {
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) {
String userID = session.getUserID();
- UserDetails userDetails = (UserDetails) authentication.getDetails();
- if (userDetails != null) {
- String currentUserName = userDetails.getUsername();
+ String currentUserName = authentication.getName();
+ if (currentUserName != null) {
if (!userID.equals(currentUserName)) {
log.warn("Current session has user ID " + userID
+ " while logged is user is " + currentUserName
+ "(authentication=" + authentication + ")"
+ ". Re-login.");
+ // TODO throw an exception
return login();
}
}
+ // 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);
}