electron-log - npm Package Compare versions

Comparing version 4.0.0-beta.5 to 4.0.0-beta.6

package.json

 {
   "name": "electron-log",
-  "version": "4.0.0-beta.5",
+  "version": "4.0.0-beta.6",
   "description": "Just a very simple logging module for your Electron application",
@@ -5,0 +5,0 @@ "main": "src/index.js",

src/log.js

@@ -7,2 +7,3 @@ 'use strict';
   runTransport: runTransport,
+  runTransports: runTransports,
 };
@@ -20,2 +21,6 @@
 
+  runTransports(transports, message, electronLog);
+}
+
+function runTransports(transports, message, electronLog) {
   for (var i in transports) {
@@ -22,0 +27,0 @@ if (Object.prototype.hasOwnProperty.call(transports, i)) {

src/transports/remote.js

@@ -6,2 +6,3 @@ 'use strict';
 var url = require('url');
+var log = require('../log');
 var transform = require('../transform');
@@ -11,3 +12,3 @@
 
-function remoteTransportFactory() {
+function remoteTransportFactory(electronLog) {
   transport.client = { name: 'electron-application' };
@@ -20,24 +21,43 @@ transport.depth = 6;
   return transport;
-}
 
-function transport(message) {
-  if (!transport.url) return;
+  function transport(message) {
+    if (!transport.url) return;
 
-  post(transport.url, {
-    client: transport.client,
-    data: transform.transform(message, [
-      transform.removeStyles,
-      transform.toJSON,
-      transform.maxDepthFactory(transport.depth + 1),
-    ]),
-    date: message.date.getTime(),
-    level: message.level,
-    styles: message.styles,
-    variables: message.variables,
-  });
+    var request = post(transport.url, transport.requestOptions, {
+      client: transport.client,
+      data: transform.transform(message, [
+        transform.removeStyles,
+        transform.toJSON,
+        transform.maxDepthFactory(transport.depth + 1),
+      ]),
+      date: message.date.getTime(),
+      level: message.level,
+      variables: message.variables,
+    });
+
+    request.on('error', function (error) {
+      var errorMessage = {
+        data: [
+          'electron-log.transports.remote:'
+          + ' cannot send HTTP request to ' + transport.url,
+          error,
+        ],
+        date: new Date(),
+        level: 'warn',
+      };
+
+      var transports = [
+        electronLog.transports.console,
+        electronLog.transports.ipc,
+        electronLog.transports.file,
+      ];
+
+      log.runTransports(transports, errorMessage, electronLog);
+    });
+  }
 }
 
-function post(serverUrl, data) {
+function post(serverUrl, requestOptions, data) {
   var urlObject = url.parse(serverUrl);
-  var htTransport = urlObject.protocol === 'https:' ? https : http;
+  var httpTransport = urlObject.protocol === 'https:' ? https : http;
 
@@ -57,7 +77,9 @@ var body = JSON.stringify(data);
 
-  Object.assign(options, transport.requestOptions);
+  Object.assign(options, requestOptions);
 
-  var request = htTransport.request(options);
+  var request = httpTransport.request(options);
   request.write(body);
   request.end();
+
+  return request;
 }

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

Total package byte prevSize

53870

increased by1.42%

Lines of code

1602

increased by1.39%

No dependency changes