Check login with an info call

pynetgear/__init__.py

@@ -101,6 +101,9 @@ def __init__(
         self.force_login_v1 = force_login_v1
         self.cookie = None
         self.config_started = False
+        self._logging_in = False
+
+        self._info = None
 
     def login(self):
         """
@@ -110,14 +113,22 @@ def login(self):
         """
         # cookie is also used to track if at least
         # one login attempt has been made for v1
+        if self._logging_in:
+            _LOGGER.debug("Login re-attempt within the login, ignoring.")
+            return False
+
+        self._logging_in = True
         self.cookie = None
 
         if not self.force_login_v1:
             v2_result = self.login_v2()
             if v2_result:
+                self._logging_in = False
                 return v2_result
 
-        return self.login_v1()
+        v1_result = self.login_v1()
+        self._logging_in = False
+        return v1_result
 
     def login_v2(self):
         _LOGGER.debug("Login v2")
@@ -143,6 +154,10 @@ def login_v2(self):
             _LOGGER.debug(response.headers)
             return False
 
+        # check login succes with info call
+        if self.get_info(use_cache=False) is None:
+            return False
+
         return True
 
     def login_v1(self):
@@ -161,9 +176,13 @@ def login_v1(self):
 
         self.cookie = success
 
+        # check login succes with info call
+        if self.get_info(use_cache=False) is None:
+            return False
+
         return success
 
-    def get_info(self):
+    def get_info(self, use_cache=True):
         """
         Return router informations, like:
         - ModelName
@@ -179,6 +198,10 @@ def get_info(self):
         """
         _LOGGER.debug("Get Info")
 
+        if self._info is not None and use_cache:
+            _LOGGER.debug("Info from cache.")
+            return self._info
+
         success, response = self._make_request(
             SERVICE_DEVICE_INFO,
             "GetInfo"
@@ -192,7 +215,9 @@ def get_info(self):
         if not success:
             return None
 
-        return {t.tag: t.text for t in node}
+        self._info = {t.tag: t.text for t in node}
+
+        return self._info
 
     def get_attached_devices(self):
         """