Changeset 3344 in josm for trunk/src/org/openstreetmap/josm/io/auth/JosmPreferencesCredentialManager.java

Timestamp:

2010-06-26T21:09:24+02:00 (14 years ago)

Author:

bastiK

Message:

fixed #4259 - Authentication dialog appears several times while uploading a changeset

File:

• trunk/src/org/openstreetmap/josm/io/auth/JosmPreferencesCredentialManager.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/io/auth/JosmPreferencesCredentialManager.java

@@ -4 +4 @@
 import java.net.PasswordAuthentication;
 import java.net.Authenticator.RequestorType;
+import java.util.HashMap;
+import java.util.Map;
 
 import org.openstreetmap.josm.Main;
@@ -17 +19 @@
 public class JosmPreferencesCredentialManager implements CredentialsManager {
 
+    Map<RequestorType, PasswordAuthentication> memoryCredentialsCache = new HashMap<RequestorType, PasswordAuthentication>();
     /**
      * @see CredentialsManager#lookup(RequestorType)
@@ -80 +83 @@
         CredentialsManagerResponse response = new CredentialsManagerResponse();
 
-        if (noSuccessWithLastResponse|| username.equals("") || password.equals("")) {
+        /*
+         * Last request was successful and there was no credentials stored
+         * in file. -> Try to recall credentials that have been entered
+         * manually in this session.
+         */
+        if (!noSuccessWithLastResponse && credentials == null && memoryCredentialsCache.containsKey(requestorType)) {
+            PasswordAuthentication pa = memoryCredentialsCache.get(requestorType);
+            response.setUsername(pa.getUserName());
+            response.setPassword(pa.getPassword());
+            response.setCanceled(false);
+        /*
+         * Prompt the user for credentials. This happens the first time each
+         * josm start if the user does not save the credentials to preference
+         * file (username=="") and each time after authentication failed
+         * (noSuccessWithLastResponse == true).
+         */
+        } else if (noSuccessWithLastResponse || username.equals("") || password.equals("")) {
             CredentialDialog dialog = null;
             switch(requestorType) {
@@ -97 +116 @@
                         response.getPassword()
                 ));
+            /*
+             * User decides not to save credentials to file. Keep it
+             * in memory so we don't have to ask over and over again.
+             */
+            } else {
+                PasswordAuthentication pa = new PasswordAuthentication(dialog.getUsername(), dialog.getPassword());
+                memoryCredentialsCache.put(requestorType, pa);
             }
+        /*
+         * We got it from file.
+         */
         } else {
             response.setUsername(username);