From c9dd2b73392d9a81785189e973e318405f18584e Mon Sep 17 00:00:00 2001 From: Danny Date: Fri, 6 Nov 2026 10:53:27 -0800 Subject: [PATCH] Add Logging for Screen Failure Response --- openpilot/starpilot/system/starport/bootstrap.py | 2 +- openpilot/starpilot/system/starport/tests/test_starport.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/openpilot/starpilot/system/starport/bootstrap.py b/openpilot/starpilot/system/starport/bootstrap.py index e31d014efb..e6419f6fb3 100644 --- a/openpilot/starpilot/system/starport/bootstrap.py +++ b/openpilot/starpilot/system/starport/bootstrap.py @@ -471,7 +471,7 @@ class WirelessBootstrap: endpoint = parsed self._reply_accepted = True else: - self.log("bootstrap_start_refused", status=status, endpoint=parsed is not None) + self.log("bootstrap_start_refused", status=status, endpoint=parsed is not None, hex=payload.hex()) self.refusals.append(status) deadline = min(latest_deadline, max(deadline, time.monotonic() + self.stage_timeout)) # a car that answers is not gone start_request_at = time.monotonic() + self.start_request_delay diff --git a/openpilot/starpilot/system/starport/tests/test_starport.py b/openpilot/starpilot/system/starport/tests/test_starport.py index c571f5e683..9d15705a1c 100644 --- a/openpilot/starpilot/system/starport/tests/test_starport.py +++ b/openpilot/starpilot/system/starport/tests/test_starport.py @@ -176,7 +176,7 @@ def test_bootstrap_waits_out_a_busy_start_response(): result, joined, _, events = run_bootstrap(start_request_delay=0.1, version_first=True, wait_for_phone_start=True, answer_start=True, busy_replies=1, busy_follow_up=True) assert result.endpoint.port == 5288 and joined[0].ssid == "HondaAA" - assert ("bootstrap_start_refused", {"status": -1, "endpoint": False}) in events + assert ("bootstrap_start_refused", {"status": -1, "endpoint": False, "hex": "18ffffffffffffffffff01"}) in events def test_bootstrap_asks_again_after_a_busy_start_response():