Improve exception messages on credential mismatches (#1417)

rytilahti · web-flow · commit e097b45984db · 2025-01-04T10:06:26.000Z
diff --git a/kasa/transports/klaptransport.py b/kasa/transports/klaptransport.py
@@ -214,8 +214,8 @@ async def perform_handshake1(self) -> tuple[bytes, bytes, bytes]:
 
             if default_credentials_seed_auth_hash == server_hash:
                 _LOGGER.debug(
-                    "Server response doesn't match our expected hash on ip %s, "
-                    "but an authentication with %s default credentials matched",
+                    "Device response did not match our expected hash on ip %s,"
+                    "but an authentication with %s default credentials worked",
                     self._host,
                     key,
                 )
@@ -235,13 +235,16 @@ async def perform_handshake1(self) -> tuple[bytes, bytes, bytes]:
 
             if blank_seed_auth_hash == server_hash:
                 _LOGGER.debug(
-                    "Server response doesn't match our expected hash on ip %s, "
-                    "but an authentication with blank credentials matched",
+                    "Device response did not match our expected hash on ip %s, "
+                    "but an authentication with blank credentials worked",
                     self._host,
                 )
                 return local_seed, remote_seed, self._blank_auth_hash  # type: ignore
 
-        msg = f"Server response doesn't match our challenge on ip {self._host}"
+        msg = (
+            f"Device response did not match our challenge on ip {self._host}, "
+            f"check that your e-mail and password (both case-sensitive) are correct. "
+        )
         _LOGGER.debug(msg)
         raise AuthenticationError(msg)
 
diff --git a/kasa/transports/sslaestransport.py b/kasa/transports/sslaestransport.py
@@ -603,7 +603,10 @@ async def perform_handshake1(self) -> tuple[str, str, str] | None:
             _LOGGER.debug("Credentials match")
             return local_nonce, server_nonce, pwd_hash
 
-        msg = f"Server response doesn't match our challenge on ip {self._host}"
+        msg = (
+            f"Device response did not match our challenge on ip {self._host}, "
+            f"check that your e-mail and password (both case-sensitive) are correct. "
+        )
         _LOGGER.debug(msg)
 
         raise AuthenticationError(msg)
