comparison easyhg.py @ 459:ee06f9275c99

Use cached password directly, without asking the user, if it's available and the requested realm/uri are not the same as an immediately preceding one

             self.auth_config.add_section(sect)
         self.auth_config.set(sect, key, data)
 
     def remote_key(self, url, user):
         # generate a "safe-for-config-file" key representing uri+user
-        self.ui.write('generating remote_key for url %s and user %s\n' % (url, user))
+#        self.ui.write('generating remote_key for url %s and user %s\n' % (url, user))
         s = '%s@@%s' % (url, user)
         h = hashlib.sha1()
         h.update(self.auth_key)
         h.update(s)
         hx = h.hexdigest()

         self.load_config()
         if not self.auth_config: return
     
         self.set_to_config('preferences', 'remember', self.remember)
 
-        self.ui.write('aiming to store details for user %s\n' % self.user)
+#        self.ui.write('aiming to store details for user %s\n' % self.user)
     
         if self.remember and self.user:
             d = self.encrypt(self.user)
             self.set_to_config('user', self.remote_user_key(), d)
         else:

     auth_store = None
 
     def __init__(self, ui, url, user, passwd):
         self.auth_store = EasyHgAuthStore(ui, url, user, passwd)
 
-    def ask(self):
+    def ask(self, force_dialog):
+
+        if self.auth_store.user and self.auth_store.passwd:
+            if not force_dialog:
+                return (self.auth_store.user, self.auth_store.passwd)
+
         dialog = QtGui.QDialog()
         layout = QtGui.QGridLayout()
         dialog.setLayout(layout)
 
         layout.addWidget(QtGui.QLabel(_('<h3>Login required</h3><p>Please provide your login details for the repository at<br><code>%s</code>:') % self.auth_store.argless_url()), 0, 0, 1, 2)

 orig_find = passwordmgr.find_user_password
 
 @monkeypatch_method(passwordmgr)
 def find_user_password(self, realm, authuri):
 
+    if not hasattr(self, '__easyhg_last'):
+        self.__easyhg_last = None
+
     if not self.ui.interactive():
         return orig_find(self, realm, authuri)
     if not easyhg_pyqt_ok:
         return orig_find(self, realm, authuri)
 
     authinfo = urllib2.HTTPPasswordMgrWithDefaultRealm.find_user_password(
         self, realm, authuri)
     user, passwd = authinfo
 
     if user and passwd:
-        self.ui.write("note: user and passwd both provided\n")
+#        self.ui.write("note: user and passwd both provided\n")
         return orig_find(self, realm, authuri)
 
-    self.ui.write("want username and/or password for %s\n" % authuri)
+#    self.ui.write("want username and/or password for %s\n" % authuri)
+
+    force_dialog = False
+
+#    if self.__easyhg_last:
+#        self.ui.write("last = realm '%s' authuri '%s'\n" % self.__easyhg_last)
+#        self.ui.write("me   = realm '%s' authuri '%s'\n" % (realm, authuri))
+
+    if (realm, authuri) == self.__easyhg_last:
+        # If we are called again just after identical previous
+        # request, then the previously returned auth must have been
+        # wrong. So we note this to force password prompt (and avoid
+        # reusing bad password indefinitely). Thanks to
+        # mercurial_keyring (Marcin Kasperski) for this logic
+#        self.ui.write("same again!\n")
+        force_dialog = True
 
     dialog = EasyHgAuthDialog(self.ui, authuri, user, passwd)
 
-    (user, passwd) = dialog.ask()
-
-    self.add_password(realm, authuri, user, passwd)
+    (user, passwd) = dialog.ask(force_dialog)
+
+#    self.add_password(realm, authuri, user, passwd)
+    self.__easyhg_last = (realm, authuri)
+#    self.ui.write("done = realm '%s' authuri '%s'\n" % self.__easyhg_last)
     return (user, passwd)