pending-dns 1.2.2 → 1.2.3

package/lib/dns-handler.js

@@ -341,10 +341,11 @@ const dnsHandler = async request => {
 
     await Promise.all(
         request.questions.map(question => {
-            logger.info({
+            logger.debug({
                 id: request._id,
                 msg: 'DNS query',
-                type: request.source.type,
+                action: 'dns_query',
+                proto: request.source.type,
                 port: request.source.port,
                 address: request.source.address,
                 question: question.name,
@@ -354,9 +355,10 @@ const dnsHandler = async request => {
         })
     );
 
-    logger.info({
-        msg: 'DNS response',
+    logger.debug({
         id: request._id,
+        msg: 'DNS response',
+        action: 'dns_response',
         dnsTime: Date.now() - startTime,
         questions: request.questions,
         answers: response.answers

package/lib/dns-server.js

@@ -59,7 +59,11 @@ const init = async () => {
             logger.info({ msg: 'DNS server listening', protocol: 'tcp', host: config.dns.host, port: config.dns.port });
         })
         .on('error', err => {
-            logger.error({ msg: 'DNS server error', protocol: 'tcp', err });
+            let method = 'error';
+            if (err && ['ECONNRESET'].includes(err.code)) {
+                method = 'trace';
+            }
+            logger[method]({ msg: 'DNS server error', protocol: 'tcp', err });
         });
 };
 

package/lib/dns-tcp-server.js

@@ -40,7 +40,7 @@ class DNSTcpServer extends EventEmitter {
                 try {
                     request = Packet.parse(buffer.slice(2));
                 } catch (err) {
-                    logger.error({ msg: 'Failed to parse DNS package', type: 'tcp', err, buffer, port: socket.remotePort, address: socket.remoteAddress });
+                    logger.trace({ msg: 'Failed to parse DNS package', proto: 'tcp', err, buffer, port: socket.remotePort, address: socket.remoteAddress });
                     try {
                         socket.end();
                     } catch (err) {
@@ -50,7 +50,7 @@ class DNSTcpServer extends EventEmitter {
                 }
 
                 request.source = {
-                    type: 'tcp',
+                    proto: 'tcp',
                     port: socket.remotePort,
                     address: socket.remoteAddress
                 };
@@ -95,10 +95,16 @@ class DNSTcpServer extends EventEmitter {
             try {
                 let len = Buffer.alloc(2);
                 len.writeUInt16BE(message.length);
-                socket.end(Buffer.concat([len, message]));
+
+                if (['open', 'writeOnly'].includes(socket.readyState)) {
+                    socket.end(Buffer.concat([len, message]));
+                } else {
+                    socket.end();
+                }
             } catch (err) {
                 // ignore
             }
+
             resolve(message);
         });
     }

package/lib/dns-udp-server.js

@@ -26,15 +26,16 @@ class DNSUdpServer extends EventEmitter {
         try {
             request = Packet.parse(buffer);
         } catch (err) {
-            logger.error({ msg: 'Failed to parse DNS package', type: 'udp', err, buffer, port: rinfo.port, address: rinfo.address });
+            logger.trace({ msg: 'Failed to parse DNS package', proto: 'udp', err, buffer, port: rinfo.port, address: rinfo.address });
             return;
         }
 
         request.source = {
-            type: 'udp',
+            proto: 'udp',
             port: rinfo.port,
             address: rinfo.address
         };
+
         this.emit('request', request, this.send.bind(this, rinfo), rinfo);
     }
     send(rinfo, message) {

package/lib/public-server.js

@@ -263,7 +263,17 @@ const setupHttps = () =>
 
                     logger.error({ msg: 'Failed to serve redirect page', err, hostname });
 
-                    const url = new URL(req.url || req.headers[':path'], `https://${hostname}/`);
+                    let url;
+                    try {
+                        url = new URL(req.url || req.headers[':path'], `https://${hostname}/`);
+                    } catch (err) {
+                        try {
+                            url = new URL(req.headers[':path'], `https://${hostname}/`);
+                        } catch (err) {
+                            url = `https://${hostname}/`;
+                        }
+                    }
+
                     const route = url.pathname;
 
                     res.send(
@@ -332,8 +342,17 @@ const setupHttp = () =>
                 const hostname = getHostname(req).replace(/^www\./, '');
 
                 logger.error({ msg: 'Failed to serve redirect page', err, hostname });
+                let url;
+                try {
+                    url = new URL(req.url || req.headers[':path'], `https://${hostname}/`);
+                } catch (err) {
+                    try {
+                        url = new URL(req.headers[':path'], `https://${hostname}/`);
+                    } catch (err) {
+                        url = `https://${hostname}/`;
+                    }
+                }
 
-                const url = new URL(req.url || req.headers[':path'], `https://${hostname}/`);
                 const route = url.pathname;
 
                 res.send(

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "pending-dns",
-    "version": "1.2.2",
+    "version": "1.2.3",
     "description": "Lightweight API driven DNS server",
     "main": "index.js",
     "scripts": {