Add stream_rtsp_url to camera module (#1197)

kasa/experimental/modules/camera.py

@@ -2,10 +2,15 @@
 
 from __future__ import annotations
 
+from urllib.parse import quote_plus
+
+from ...credentials import Credentials
 from ...device_type import DeviceType
 from ...feature import Feature
 from ..smartcameramodule import SmartCameraModule
 
+LOCAL_STREAMING_PORT = 554
+
 
 class Camera(SmartCameraModule):
     """Implementation of device module."""
@@ -31,11 +36,32 @@ class Camera(SmartCameraModule):
     @property
     def is_on(self) -> bool:
         """Return the device id."""
-        return self.data["lens_mask_info"]["enabled"] == "on"
+        return self.data["lens_mask_info"]["enabled"] == "off"
+
+    def stream_rtsp_url(self, credentials: Credentials | None = None) -> str | None:
+        """Return the local rtsp streaming url.
+
+        :param credentials: Credentials for camera account.
+            These could be different credentials to tplink cloud credentials.
+            If not provided will use tplink credentials if available
+        :return: rtsp url with escaped credentials or None if no credentials or
+            camera is off.
+        """
+        if not self.is_on:
+            return None
+        dev = self._device
+        if not credentials:
+            credentials = dev.credentials
+        if not credentials or not credentials.username or not credentials.password:
+            return None
+        username = quote_plus(credentials.username)
+        password = quote_plus(credentials.password)
+        return f"rtsp://{username}:{password}@{dev.host}:{LOCAL_STREAMING_PORT}/stream1"
 
     async def set_state(self, on: bool) -> dict:
         """Set the device state."""
-        params = {"enabled": "on" if on else "off"}
+        # Turning off enables the privacy mask which is why value is reversed.
+        params = {"enabled": "off" if on else "on"}
         return await self._device._query_setter_helper(
             "setLensMaskConfig", self.QUERY_MODULE_NAME, "lens_mask_info", params
         )

kasa/experimental/sslaestransport.py

@@ -120,11 +120,13 @@ class SslAesTransport(BaseTransport):
         self._seq: int | None = None
         self._pwd_hash: str | None = None
         self._username: str | None = None
+        self._password: str | None = None
         if self._credentials != Credentials() and self._credentials:
             self._username = self._credentials.username
+            self._password = self._credentials.password
         elif self._credentials_hash:
             ch = json_loads(base64.b64decode(self._credentials_hash.encode()))
-            self._pwd_hash = ch["pwd"]
+            self._password = ch["pwd"]
             self._username = ch["un"]
         self._local_nonce: str | None = None
 
@@ -140,10 +142,10 @@ class SslAesTransport(BaseTransport):
         """The hashed credentials used by the transport."""
         if self._credentials == Credentials():
             return None
-        if self._credentials_hash:
+        if not self._credentials and self._credentials_hash:
             return self._credentials_hash
-        if self._pwd_hash and self._credentials:
-            ch = {"un": self._credentials.username, "pwd": self._pwd_hash}
+        if (cred := self._credentials) and cred.password and cred.username:
+            ch = {"un": cred.username, "pwd": cred.password}
             return base64.b64encode(json_dumps(ch).encode()).decode()
         return None