Start mdns inside of a try/catch to ensure that the app will start wi…

…thout mdns
prgsmall · Apr 18, 2012 · f78a4f0 · f78a4f0
1 parent cb7de0f
commit f78a4f0
Showing 1 changed file with 7 additions and 11 deletions.
diff --git a/lib/acequia.js b/lib/acequia.js
@@ -20,8 +20,6 @@ var INTERNAL_IP = "0.0.0.0",
 
 var tcp_ad, osc_ad, ws_ad;
 
-var isWindows = !!require("os").platform().match(/^win/);
-
 // The list of clients
 var clients = new ac.AcequiaClients(TIMEOUT * 1000);
 
@@ -58,13 +56,12 @@ function startServers() {
     oscServer.on("listening", function () {
         logger.debug("OSC Server is listening on [%s:%s]", oscServer.address().address, oscServer.address().port);
 
-        if (!isWindows) {
-        	try {
+        try {
             var mdns = require("mdns"),
             osc_ad = mdns.createAdvertisement(mdns.udp('acequia', 'osc'), oscServer.address().port, {name: "Acequia OSC Server"});
             osc_ad.start();
-            }catch(e){
-            }
+        } catch (e) {
+            logger.error("Error creating mDNS advertisement: " + e.message);
         }
     });
 
@@ -174,17 +171,16 @@ function startServers() {
             socket.destroy();
         });
     });
-    
+
     tcpServer.on("listening", function () {
         logger.debug("TCP Server is listening on [%s:%s]", tcpServer.address().address, tcpServer.address().port);
 
-        if (!isWindows) {
-        	try {
+        try {
             var mdns = require("mdns");
             tcp_ad = mdns.createAdvertisement(mdns.tcp('acequia'), tcpServer.address().port, {name: "Acequia TCP Server"});
             tcp_ad.start();
-            }catch(e){
-            }
+        } catch (e) {
+            logger.error("Error creating mDNS advertisement: " + e.message);
         }
     });