Alan-Jowett / sonde / 24639942947

82%

push

github

fix(gateway): warn when WAKE blob has no matching handler (#764)

* fix(gateway): warn when WAKE blob has no matching handler

The WAKE blob routing path silently dropped blobs when no handler was
registered for the program_hash.  The APP_DATA path already emits a
WARN with node_id, program_hash, and handler_count; apply the same
treatment to WAKE-originated blobs.

Also capture handler_count inside the read-lock alongside the handler
lookup so both fields are available in the else branch after the lock
is released.

Adds T-1309 to verify the warning is emitted.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* style: apply rustfmt

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* test(gateway): fix T-1309 flaky sleep and add program_hash assertion

- Replace fixed 100ms sleep with a poll loop (5s deadline, 10ms interval)
  to avoid flakiness on slow CI runners
- Assert that the WARN includes the program_hash hex, matching the doc
  comment which already listed it as a verified field

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* test(gateway): also assert program_hash= field key in T-1309

Asserting only the hex value could pass if the hash appeared in a
different log line. Add an explicit check for the structured field
key so the test is tied to the WARN's own fields.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

---------

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

8 of 8 new or added lines in 1 file covered. (100.0%)

24438 of 30246 relevant lines covered (80.8%)

203.66 hits per line