python-aidot-cameras 0.11.7__tar.gz → 0.11.9__tar.gz

{python_aidot_cameras-0.11.7/python_aidot_cameras.egg-info → python_aidot_cameras-0.11.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-aidot-cameras
-Version: 0.11.7
+Version: 0.11.9
 Summary: Control AiDot/Leedarson WiFi lights and cameras (WebRTC streaming, two-way audio, PTZ, controls)
 Author-email: cbrightly <chris.brightly@gmail.com>
 License-Expression: MIT

{python_aidot_cameras-0.11.7 → python_aidot_cameras-0.11.9}/aidot/camera/client.py

@@ -792,6 +792,16 @@ class CameraMixin(_CameraControlsMixin, _WebRTCOpenMixin, _SdesOpenMixin):
             self._cloud_online_explicit = True
         return self.status
 
+    def _open_fail_logger(self):
+        """Pick the log level for a stream-open failure in the serve/keepalive
+        loops: WARNING for a genuine failure, but DEBUG when the cloud has
+        explicitly reported the device offline.  A dead/unpowered camera is
+        already throttled to ~probe cadence by _backoff_or_offline_pause, so its
+        expected open failures should not drip WARNINGs into the log every probe.
+        """
+        _offline = (not self.status.online) and getattr(self, "_cloud_online_explicit", False)
+        return _LOGGER.debug if _offline else _LOGGER.warning
+
     async def _backoff_or_offline_pause(self, delay: float) -> None:
         """Sleep a reconnect backoff, extended while the device is cloud-offline.
 
@@ -2732,7 +2742,7 @@ class CameraMixin(_CameraControlsMixin, _WebRTCOpenMixin, _SdesOpenMixin):
                         "falling back to full relay path", self.device_id,
                         _FAST_OPEN_TIMEOUT)
                 _delay = _pacer.fail_delay()
-                _LOGGER.warning(
+                self._open_fail_logger()(
                     "SDES keepalive: stream open failed for %s (retry in %.0fs): %s",
                     self.device_id, _delay, exc,
                 )
@@ -2908,7 +2918,7 @@ class CameraMixin(_CameraControlsMixin, _WebRTCOpenMixin, _SdesOpenMixin):
                 continue
             except Exception as exc:
                 _delay = _pacer.fail_delay()
-                _LOGGER.warning(
+                self._open_fail_logger()(
                     "Stream open failed for %s (retry in %.0fs): %s",
                     self.device_id, _delay, exc,
                 )
@@ -3409,7 +3419,7 @@ class CameraMixin(_CameraControlsMixin, _WebRTCOpenMixin, _SdesOpenMixin):
                 continue
             except Exception as exc:
                 _delay = _pacer.fail_delay()
-                _LOGGER.warning(
+                self._open_fail_logger()(
                     "DTLS serve: open failed for %s (retry %.0fs): %s",
                     self.device_id, _delay, exc,
                 )

{python_aidot_cameras-0.11.7 → python_aidot_cameras-0.11.9}/aidot/camera/sdes.py

@@ -132,6 +132,29 @@ class SdesSession:
         """time.monotonic() of the last media packet forwarded, 0.0 if none yet."""
         return self._media_progress[0]
 
+    def _log_ffmpeg_stderr(self, stderr_bytes: bytes) -> None:
+        """Log ffmpeg's captured stderr at a level that matches what happened.
+
+        The common NO_MEDIA case - the camera's SRTP never arrives, so ffmpeg
+        finds no keyframe and writes an empty file - would otherwise spam a
+        WARNING on every serve retry for a camera that never delivers media.
+        Demote to debug ONLY when no media ever arrived AND the stderr is that
+        expected shape.  Anything else - media WAS flowing (a genuine mid-stream
+        error), or an unexpected ffmpeg failure with no media (malformed SDP, bad
+        SRTP key, protocol reject) - still logs at WARNING, so real problems are
+        not hidden.  Unknown ffmpeg wording degrades safely to WARNING.
+        """
+        if not stderr_bytes:
+            return
+        text = stderr_bytes.decode(errors="replace")
+        expected_no_media = self.last_media_monotonic == 0.0 and (
+            "Output file is empty" in text
+            or "Could not find codec parameters" in text
+        )
+        (_LOGGER.debug if expected_no_media else _LOGGER.warning)(
+            "ffmpeg SDES stderr:\n%s", text
+        )
+
     @staticmethod
     def is_stalled(
         last_media: float,
@@ -213,8 +236,7 @@ class SdesSession:
                 self._proc.stderr.close()
             except Exception:
                 _LOGGER.debug("swallowed exception in %s", 'stop', exc_info=True)
-        if stderr_bytes:
-            _LOGGER.warning("ffmpeg SDES stderr:\n%s", stderr_bytes.decode(errors="replace"))
+        self._log_ffmpeg_stderr(stderr_bytes)
         import os
         try:
             os.unlink(self._sdp_path)

{python_aidot_cameras-0.11.7 → python_aidot_cameras-0.11.9}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "python-aidot-cameras"
-version = "0.11.7"
+version = "0.11.9"
 description = "Control AiDot/Leedarson WiFi lights and cameras (WebRTC streaming, two-way audio, PTZ, controls)"
 readme = "README.md"
 license = "MIT"

{python_aidot_cameras-0.11.7 → python_aidot_cameras-0.11.9/python_aidot_cameras.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-aidot-cameras
-Version: 0.11.7
+Version: 0.11.9
 Summary: Control AiDot/Leedarson WiFi lights and cameras (WebRTC streaming, two-way audio, PTZ, controls)
 Author-email: cbrightly <chris.brightly@gmail.com>
 License-Expression: MIT

{python_aidot_cameras-0.11.7 → python_aidot_cameras-0.11.9}/python_aidot_cameras.egg-info/SOURCES.txt

@@ -95,6 +95,7 @@ tests/test_mqtt_session_reconnect.py
 tests/test_narrow_pc_ice.py
 tests/test_no_undefined_names.py
 tests/test_offline_keepalive_pause.py
+tests/test_open_fail_logger.py
 tests/test_open_gate_delay.py
 tests/test_persistent_mqtt.py
 tests/test_persistent_mqtt_robustness.py
@@ -104,6 +105,7 @@ tests/test_retry_policy.py
 tests/test_sdes_adaptive.py
 tests/test_sdes_echo_wait_timeout.py
 tests/test_sdes_fast_liveplay.py
+tests/test_sdes_ffmpeg_stderr_log.py
 tests/test_sdes_idle_release.py
 tests/test_sdes_open_cleanup.py
 tests/test_sdes_serve_audio.py

python_aidot_cameras-0.11.9/tests/test_open_fail_logger.py

@@ -0,0 +1,29 @@
+"""Serve/keepalive open-failure logging level.
+
+A stream-open failure logs at WARNING normally, but at DEBUG when the cloud has
+explicitly reported the device offline - a dead/unpowered camera is throttled to
+probe cadence by the offline pause and should not drip WARNINGs every probe.
+"""
+import types
+
+from aidot.camera import client as camclient
+
+
+def _dc(online, explicit):
+    o = types.SimpleNamespace()
+    o.status = types.SimpleNamespace(online=online)
+    o._cloud_online_explicit = explicit
+    return camclient.CameraMixin._open_fail_logger.__get__(o)
+
+
+def test_debug_when_cloud_reports_offline():
+    assert _dc(online=False, explicit=True)() == camclient._LOGGER.debug
+
+
+def test_warning_when_online():
+    assert _dc(online=True, explicit=True)() == camclient._LOGGER.warning
+
+
+def test_warning_when_online_state_unknown():
+    # Never explicitly reported offline -> treat a failure as genuine (WARNING).
+    assert _dc(online=False, explicit=False)() == camclient._LOGGER.warning

python_aidot_cameras-0.11.9/tests/test_sdes_ffmpeg_stderr_log.py

@@ -0,0 +1,52 @@
+"""ffmpeg SDES stderr log level: quiet the expected NO_MEDIA case, keep real errors.
+
+A camera whose SRTP never reaches us (NO_MEDIA - e.g. media not traversing the
+relay) produces the expected "could not find codec parameters / empty output"
+ffmpeg stderr on every serve retry. That must log at DEBUG (not spam WARNING).
+But a genuine ffmpeg error - whether media was flowing, or an unexpected failure
+with no media - still logs at WARNING so real problems are not hidden.
+"""
+import logging
+import types
+
+from aidot.camera.sdes import SdesSession
+
+
+def _log_fn(last_media):
+    obj = types.SimpleNamespace(last_media_monotonic=last_media)
+    return SdesSession._log_ffmpeg_stderr.__get__(obj)
+
+
+def _stderr_records(caplog):
+    return [r for r in caplog.records if "ffmpeg SDES stderr" in r.getMessage()]
+
+
+def test_no_media_expected_shape_logs_debug(caplog):
+    for msg in (b"...Output file is empty, nothing was encoded",
+                b"Could not find codec parameters for stream 1 (Video: h264, none)"):
+        caplog.clear()
+        with caplog.at_level(logging.DEBUG, logger="aidot.camera.sdes"):
+            _log_fn(0.0)(msg)
+        recs = _stderr_records(caplog)
+        assert recs and recs[0].levelno == logging.DEBUG, msg
+
+
+def test_no_media_unexpected_error_still_warns(caplog):
+    # No media, but NOT the expected empty-output shape -> a real ffmpeg failure.
+    with caplog.at_level(logging.DEBUG, logger="aidot.camera.sdes"):
+        _log_fn(0.0)(b"/tmp/x.sdp: Permission denied")
+    recs = _stderr_records(caplog)
+    assert recs and recs[0].levelno == logging.WARNING
+
+
+def test_media_flowed_logs_warning(caplog):
+    with caplog.at_level(logging.DEBUG, logger="aidot.camera.sdes"):
+        _log_fn(1234.5)(b"Could not find codec parameters for stream 1")
+    recs = _stderr_records(caplog)
+    assert recs and recs[0].levelno == logging.WARNING
+
+
+def test_empty_stderr_logs_nothing(caplog):
+    with caplog.at_level(logging.DEBUG, logger="aidot.camera.sdes"):
+        _log_fn(0.0)(b"")
+    assert _stderr_records(caplog) == []