add query response callback option for mdns

Author: drahnieR

index.js

@@ -6,9 +6,9 @@ var Browser = require('./lib/browser')
 
 module.exports = Bonjour
 
-function Bonjour (opts) {
-  if (!(this instanceof Bonjour)) return new Bonjour(opts)
-  this._server = new Server(opts)
+function Bonjour (opts, cbQueryRespond) {
+  if (!(this instanceof Bonjour)) return new Bonjour(opts, cbQueryRespond)
+  this._server = new Server(opts, cbQueryRespond)
   this._registry = new Registry(this._server)
 }
 

lib/mdns-server.js

@@ -7,11 +7,14 @@ var deepEqual = require('deep-equal')
 
 module.exports = Server
 
-function Server (opts) {
+function Server (opts, cbQueryRespond) {
   this.mdns = multicastdns(opts)
   this.mdns.setMaxListeners(0)
   this.registry = {}
   this.mdns.on('query', this._respondToQuery.bind(this))
+  this.cbQueryRespond = typeof cbQueryRespond == 'function'
+    ? cbQueryRespond
+    : function(err) { if (err) throw err }
 }
 
 Server.prototype.register = function (records) {
@@ -83,9 +86,7 @@ Server.prototype._respondToQuery = function (query) {
         })
     }
 
-    self.mdns.respond({ answers: answers, additionals: additionals }, function (err) {
-      if (err) throw err // TODO: Handle this (if no callback is given, the error will be ignored)
-    })
+    self.mdns.respond({ answers: answers, additionals: additionals }, this.cbQueryRespond)
   })
 }