From 4af73d037741d7ed180e4f156dd08df8c495c620 Mon Sep 17 00:00:00 2001 From: ZanzyTHEbar Date: Fri, 3 Mar 2023 13:04:53 +0000 Subject: [PATCH] fix: bug in MDNS handler --- ESP/lib/src/network/mDNS/MDNSManager.cpp | 8 ++++---- ESP/src/main.cpp | 5 +++-- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/ESP/lib/src/network/mDNS/MDNSManager.cpp b/ESP/lib/src/network/mDNS/MDNSManager.cpp index 2b05bc2..c267b3d 100644 --- a/ESP/lib/src/network/mDNS/MDNSManager.cpp +++ b/ESP/lib/src/network/mDNS/MDNSManager.cpp @@ -15,10 +15,10 @@ bool MDNSHandler::startMDNS() } stateManager->setState(MDNSState_e::MDNSState_Starting); - MDNS.addService(mdnsConfig->hostname.c_str(), "tcp", 80); - char port[20]; + MDNS.addService(("_" + mdnsConfig->hostname).c_str(), "_tcp", 80); + char port[20]; //! Add service needs leading _ on ESP32 implementation for some reason (according to the docs) - MDNS.addServiceTxt(("_" + mdnsConfig->hostname).c_str(), "_tcp", "stream_port", (const char *)Helpers::itoa(80, port, 10)); //! convert int to const char* using a very efficient implemenation of itoa + MDNS.addServiceTxt(("_" + mdnsConfig->hostname).c_str(), "_tcp", "stream_port", "80"); //! convert int to const char* using a very efficient implemenation of itoa log_i("MDNS initialized!"); stateManager->setState(MDNSState_e::MDNSState_Started); return true; @@ -35,4 +35,4 @@ void MDNSHandler::update(ObserverEvent::Event event) default: break; } -} \ No newline at end of file +} diff --git a/ESP/src/main.cpp b/ESP/src/main.cpp index 248b8fb..a815c39 100644 --- a/ESP/src/main.cpp +++ b/ESP/src/main.cpp @@ -50,8 +50,9 @@ void setup() deviceConfig.load(); wifiHandler._enable_adhoc = ENABLE_ADHOC; wifiHandler.setupWifi(); + mdnsHandler.startMDNS(); - mdnsStateManager.setState(MDNSState_e::MDNSState_Starting); + /* mdnsStateManager.setState(MDNSState_e::MDNSState_Starting); switch (mdnsStateManager.getCurrentState()) { case MDNSState_e::MDNSState_Starting: @@ -63,7 +64,7 @@ void setup() break; default: break; - } + } */ switch (wifiStateManager.getCurrentState()) {