Alan-Jowett / sonde / 23359996811

82%

push

github

fix(gateway): use getrandom::fill() instead of rand::rng() for starting_seq (#390)

* fix(gateway): use getrandom::fill() instead of rand::rng() for starting_seq

Replace rand::rng().random() with getrandom::fill() in the WAKE
handler starting_seq generation to ensure cryptographic randomness
comes from the OS CSPRNG.

- Remove rand crate dependency (no remaining usages)
- Remove use rand::RngExt import from engine.rs

Closes #381

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: handle CSPRNG failure gracefully in WAKE handler

Replace .expect() with recoverable error handling for getrandom::fill()
in handle_wake(), matching the existing pattern used in ble_pairing.rs
and gateway_identity.rs. Returns None instead of panicking the gateway.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

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

5 of 7 new or added lines in 1 file covered. (71.43%)

16529 of 20004 relevant lines covered (82.63%)

49.65 hits per line