ahoy icon indicating copy to clipboard operation
ahoy copied to clipboard

[Bug] MDNS broken since 0.8.98

Open fsck-block opened this issue 1 year ago • 1 comments
trafficstars

Beim merge von "new network" in v. 0.8.98 wurde das MDNS feature nicht vollständig übernommen.

Für die Funktion von MDNS ist es notwendig dass regelmäßig die Funktion MDNS.update() aufgerufen wird. Bis zum o.g. merge war dieser Aufruf in ahoywifi::tickWifiLoop() zu finden.

Desweiteren scheint der Ticker everySec(std::bind(&AhoyNetwork::tickNetworkLoop, mNetwork), "net"); nicht mehr dauerhaft zu funktionieren, da er nicht bei den app::regularTickers() mit dabei ist.

Ein Lösung wäre den "net" ticker in die app::regularTickers() aufzunehmen und den Aufruf von MDNS.update() in AhoyWifi::tickNetworkLoop case NetworkState::GOT_IP: aufzunehmen.

Falls gewünscht kann ich einen PR beisteuern.

fsck-block avatar Mar 31 '24 15:03 fsck-block

danke für den Hinweis, habe es gerade eingebaut und werde es mit der nächsten Version veröffentlichen

lumapu avatar Apr 01 '24 11:04 lumapu