X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms.ui%2Fsrc%2Forg%2Fargeo%2Fcms%2Fwidgets%2Fauth%2FCmsLogin.java;h=db0ae6296c5bbf9e68655721f26e416108ee183c;hb=f9ee9620626e471a99f25e84175e27380d902957;hp=1809998070bc5e2a724a90b34b4e02a58cfd93fb;hpb=7e333073d07b780efe681306a1842a750cbea83c;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms.ui/src/org/argeo/cms/widgets/auth/CmsLogin.java b/org.argeo.cms.ui/src/org/argeo/cms/widgets/auth/CmsLogin.java
index 180999807..db0ae6296 100644
--- a/org.argeo.cms.ui/src/org/argeo/cms/widgets/auth/CmsLogin.java
+++ b/org.argeo.cms.ui/src/org/argeo/cms/widgets/auth/CmsLogin.java
@@ -7,20 +7,19 @@ import java.io.IOException;
 import java.util.List;
 import java.util.Locale;
 
+import javax.security.auth.Subject;
 import javax.security.auth.callback.Callback;
 import javax.security.auth.callback.CallbackHandler;
 import javax.security.auth.callback.LanguageCallback;
 import javax.security.auth.callback.NameCallback;
 import javax.security.auth.callback.PasswordCallback;
 import javax.security.auth.callback.UnsupportedCallbackException;
-import javax.security.auth.login.FailedLoginException;
 import javax.security.auth.login.LoginContext;
 import javax.security.auth.login.LoginException;
 
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.argeo.cms.CmsMsg;
-import org.argeo.cms.auth.CurrentUser;
 import org.argeo.cms.auth.HttpRequestCallback;
 import org.argeo.cms.i18n.LocaleUtils;
 import org.argeo.cms.ui.CmsStyles;
@@ -59,6 +58,9 @@ public class CmsLogin implements CmsStyles, CallbackHandler {
 
 	private final CmsView cmsView;
 
+	// optional subject to be set explicitly
+	private Subject subject = null;
+
 	public CmsLogin(CmsView cmsView) {
 		this.cmsView = cmsView;
 		defaultLocale = Activator.getNodeState().getDefaultLocale();
@@ -80,7 +82,7 @@ public class CmsLogin implements CmsStyles, CallbackHandler {
 	}
 
 	protected boolean isAnonymous() {
-		return CurrentUser.isAnonymous(cmsView.getLoginContext().getSubject());
+		return cmsView.isAnonymous();
 	}
 
 	public final void createUi(Composite parent) {
@@ -151,7 +153,6 @@ public class CmsLogin implements CmsStyles, CallbackHandler {
 
 		Integer textWidth = 120;
 		CmsUtils.style(parent, CMS_USER_MENU);
-
 		// new Label(this, SWT.NONE).setText(CmsMsg.username.lead());
 		usernameT = new Text(credentialsBlock, SWT.BORDER);
 		usernameT.setMessage(username.lead(locale));
@@ -190,8 +191,7 @@ public class CmsLogin implements CmsStyles, CallbackHandler {
 	}
 
 	/**
-	 * To be overridden in order to provide custome login button and other
-	 * links.
+	 * To be overridden in order to provide custom login button and other links.
 	 */
 	protected void extendsCredentialsBlock(Composite credentialsBlock, Locale selectedLocale,
 			SelectionListener loginSelectionListener) {
@@ -246,17 +246,28 @@ public class CmsLogin implements CmsStyles, CallbackHandler {
 	}
 
 	protected boolean login() {
+		// TODO use CmsVie in order to retrieve subject?
 		// Subject subject = cmsView.getLoginContext().getSubject();
-		LoginContext loginContext = cmsView.getLoginContext();
+		// LoginContext loginContext = cmsView.getLoginContext();
 		try {
 			//
 			// LOGIN
 			//
-			loginContext.logout();
-			loginContext = new LoginContext(NodeConstants.LOGIN_CONTEXT_USER, this);
+			// loginContext.logout();
+			LoginContext loginContext;
+			if (subject == null)
+				loginContext = new LoginContext(NodeConstants.LOGIN_CONTEXT_USER, this);
+			else
+				loginContext = new LoginContext(NodeConstants.LOGIN_CONTEXT_USER, subject, this);
 			loginContext.login();
-		} catch (FailedLoginException e) {
-			log.warn(e.getMessage());
+			cmsView.authChange(loginContext);
+			return true;
+		} catch (LoginException e) {
+			if (log.isTraceEnabled())
+				log.warn("Login failed: " + e.getMessage(), e);
+			else
+				log.warn("Login failed: " + e.getMessage());
+
 			try {
 				Thread.sleep(3000);
 			} catch (InterruptedException e2) {
@@ -264,12 +275,11 @@ public class CmsLogin implements CmsStyles, CallbackHandler {
 			}
 			// ErrorFeedback.show("Login failed", e);
 			return false;
-		} catch (LoginException e) {
-			log.error("Cannot login", e);
-			return false;
 		}
-		cmsView.authChange(loginContext);
-		return true;
+		// catch (LoginException e) {
+		// log.error("Cannot login", e);
+		// return false;
+		// }
 	}
 
 	protected void logout() {
@@ -284,11 +294,27 @@ public class CmsLogin implements CmsStyles, CallbackHandler {
 				((NameCallback) callback).setName(usernameT.getText());
 			else if (callback instanceof PasswordCallback && passwordT != null)
 				((PasswordCallback) callback).setPassword(passwordT.getTextChars());
-			else if (callback instanceof HttpRequestCallback)
+			else if (callback instanceof HttpRequestCallback) {
 				((HttpRequestCallback) callback).setRequest(UiContext.getHttpRequest());
-			else if (callback instanceof LanguageCallback && localeChoice != null)
-				((LanguageCallback) callback).setLocale(localeChoice.getSelectedLocale());
+				((HttpRequestCallback) callback).setResponse(UiContext.getHttpResponse());
+			} else if (callback instanceof LanguageCallback) {
+				Locale toUse = null;
+				if (localeChoice != null)
+					toUse = localeChoice.getSelectedLocale();
+				else if (defaultLocale != null)
+					toUse = defaultLocale;
+
+				if (toUse != null) {
+					((LanguageCallback) callback).setLocale(toUse);
+					UiContext.setLocale(toUse);
+				}
+
+			}
 		}
 	}
 
+	public void setSubject(Subject subject) {
+		this.subject = subject;
+	}
+
 }