gelf-pro - npm Package Compare versions

Comparing version 1.3.9 to 1.3.10

lib/adapter/udp.js

@@ -102,3 +102,4 @@ /**
 
-    var packetId = Array.prototype.slice.call(crypto.randomBytes(8));
+    var packetId = Array.prototype.slice.call(crypto.randomBytes(8)),
+      chunksSent = 0;
     for (var idx in chunks) {
@@ -114,12 +115,10 @@ if (isInterrupted) {
         packet, 0, packet.length, self.options.port, self.options.host,
-        function (chunkIdx) {
-          return function (err, bytesSent) {
-            if (err) { return cbResults(err, 0); }
-            bytesSentTotal += bytesSent;
-            /* istanbul ignore else */
-            if (chunkIdx >= chunksCount - 1) {
-              cbResults(err, bytesSentTotal);
-            }
+        function (err, bytesSent) {
+          if (err) { return cbResults(err, 0); }
+          bytesSentTotal += bytesSent;
+          /* istanbul ignore else */
+          if (++chunksSent >= chunksCount) {
+            cbResults(err, bytesSentTotal);
           }
-        }(idx)
+        }
       );
@@ -126,0 +125,0 @@ }

package.json

 {
   "name": "gelf-pro",
-  "version": "1.3.9",
+  "version": "1.3.10",
   "main": "./lib/gelf-pro.js",
@@ -5,0 +5,0 @@ "typings": "./typings/index.d.ts",

README.md

@@ -16,3 +16,3 @@ gelf pro
 
-`npm install gelf-pro` (**ALL** node.js versions are supported [0.x to 18.x] :)
+`npm install gelf-pro` (**ALL** node.js versions are supported [0.x to 19.x] :)
 
@@ -28,4 +28,4 @@ Library depends on: `lodash#~4.17`
 
-To provide predictable behaviour, all adapters do **NOT** re-use the socket connection.
-There are multiple ([1](https://github.com/kkamkou/node-gelf-pro/pull/68), [2](https://github.com/fdelayen/node-gelf-pro/commit/b52b4b6b1ff26772314b8673dd6fd724c0937caa)) variants available which you can borrow from (and create a new adapter).
+> :warning: To provide predictable behaviour, all existing adapters do **NOT** re-use the socket connection.  
+There are multiple ([1](https://github.com/kkamkou/node-gelf-pro/pull/68), [2](https://github.com/fdelayen/node-gelf-pro/commit/b52b4b6b1ff26772314b8673dd6fd724c0937caa)) variants available which you can borrow from (and create a new adapter).  
 [See related section](#third-party-adapters).
@@ -41,4 +41,5 @@
 
-> *(hint)* withing more or less stable network (which is most likely) I would recommend using the "udp" adapter.    
-I would also recommend it for an average to high-loaded projects. For sensitive information "tcp-tls" adapter is recommended.
+> Withing more or less stable network (which is most likely) I would recommend using the "udp" adapter.    
+I would also recommend it for an average to high-loaded project.  
+For sensitive information `tcp-tls` adapter is recommended.
 
@@ -45,0 +46,0 @@ ### Configuration

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Number of lines in readme file

194

increased by0.52%

Worsened metrics

Total package byte prevSize

24463

decreased by-0.11%

Lines of code

531

decreased by-0.19%

No dependency changes