Merge pull request log4js-node#395 from dcolens/dcolens-patch-1

nomiddlename · web-flow · commit defeb2503be0 · 2016-07-17T17:36:08.000+10:00
Fixes log4js-node#393 - catch error when sending UDP packet to GELF server
diff --git a/lib/appenders/gelf.js b/lib/appenders/gelf.js
@@ -114,9 +114,9 @@ function gelfAppender (layout, host, port, hostname, facility) {
   }
 
   function sendPacket(packet) {
-    try {
-      client.send(packet, 0, packet.length, port, host);
-    } catch(e) {}
+    client.send(packet, 0, packet.length, port, host, function(err) {
+      if (err) { console.error(err); }
+    });
   }
 
   return function(loggingEvent) {

Original file line number	Diff line number	Diff line change
`@@ -114,9 +114,9 @@ function gelfAppender (layout, host, port, hostname, facility) {`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`function sendPacket(packet) {`
`117`		`- try {`
`118`		`- client.send(packet, 0, packet.length, port, host);`
`119`		`- } catch(e) {}`
	`117`	`+ client.send(packet, 0, packet.length, port, host, function(err) {`
	`118`	`+ if (err) { console.error(err); }`
	`119`	`+ });`
`120`	`120`	`}`
`121`	`121`
`122`	`122`	`return function(loggingEvent) {`