dangernoodle-io / breadboard / 24905680052

100%

push

github

fix(mdns): bye-on-shutdown, disconnect teardown, txt refresh, mac suffix (#77)

operators were seeing stale taipanminer-2/-3/-4 service instances
accumulate in neighbor caches and .local hostnames time out after
reboots. four fixes:

1. bb_wifi: new bb_wifi_register_on_disconnect(cb) api so other
   components can hook the disconnect path cleanly
2. bb_mdns: mac-suffixed default instance name ("TaipanMiner" on the
   app side becomes "TaipanMiner-acbc" on the wire) so devices never
   probe-collide with their own stale cached records; the esp-idf
   mdns library's -N mangling stops firing
3. bb_mdns: clean teardown on wifi disconnect (mdns_service_remove_all
   + mdns_free); re-init on next got_ip via the existing callback.
   stops mdns task from running with stale state after a disconnect
4. bb_mdns: esp_register_shutdown_handler emits bye packets
   (mdns_service_remove_all + 100ms flush delay + mdns_free) on clean
   reboot so neighbors drop the record immediately instead of waiting
   for the 120s ttl
5. bb_mdns: bb_mdns_set_txt(key, value) public api so apps can refresh
   txt records after ota (version) and settings changes (worker,
   pool). previously txt was set-once at init and never updated.

81 of 81 branches covered (100.0%)

Branch coverage included in aggregate %.

113 of 113 relevant lines covered (100.0%)

274.59 hits per line