Printing process warning's via the appropriate logger

Author: kobelb

bin/kibana

@@ -21,4 +21,4 @@ if [ ! -x "$NODE" ]; then
   exit 1
 fi
 
-exec "${NODE}" $NODE_OPTIONS "${DIR}/src/cli" ${@}
+exec "${NODE}" $NODE_OPTIONS --no-warnings "${DIR}/src/cli" ${@}

bin/kibana-plugin

@@ -21,4 +21,4 @@ if [ ! -x "$NODE" ]; then
   exit 1
 fi
 
-exec "${NODE}" $NODE_OPTIONS "${DIR}/src/cli_plugin" ${@}
+exec "${NODE}" $NODE_OPTIONS --no-warnings "${DIR}/src/cli_plugin" ${@}

bin/kibana-plugin.bat

@@ -22,7 +22,7 @@ If Not Exist "%NODE%" (
 )
 
 TITLE Kibana Server
-"%NODE%" %NODE_OPTIONS% "%DIR%\src\cli_plugin" %*
+"%NODE%" %NODE_OPTIONS% --no-warnings "%DIR%\src\cli_plugin" %*
 
 :finally
 

bin/kibana.bat

@@ -22,7 +22,7 @@ If Not Exist "%NODE%" (
 )
 
 TITLE Kibana Server
-"%NODE%" %NODE_OPTIONS% "%DIR%\src\cli" %*
+"%NODE%" %NODE_OPTIONS% --no-warnings "%DIR%\src\cli" %*
 
 :finally
 

src/cli_plugin/install/index.js

@@ -6,6 +6,7 @@ import pkg from '../../utils/package_json';
 import { getConfig } from '../../server/path';
 import { parse, parseMilliseconds } from './settings';
 import { find } from 'lodash';
+import logWarnings from '../lib/log_warnings';
 
 function processCommand(command, options) {
   let settings;
@@ -18,6 +19,7 @@ function processCommand(command, options) {
   }
 
   const logger = new Logger(settings);
+  logWarnings(settings, logger);
   install(settings, logger);
 }
 

src/cli_plugin/lib/log_warnings.js

@@ -0,0 +1,5 @@
+export default function (settings, logger) {
+  process.on('warning', (warning) => {
+    logger.error(warning);
+  });
+}

src/cli_plugin/list/index.js

@@ -2,6 +2,7 @@ import { fromRoot } from '../../utils';
 import list from './list';
 import Logger from '../lib/logger';
 import { parse } from './settings';
+import logWarnings from '../lib/log_warnings';
 
 function processCommand(command, options) {
   let settings;
@@ -14,6 +15,7 @@ function processCommand(command, options) {
   }
 
   const logger = new Logger(settings);
+  logWarnings(settings, logger);
   list(settings, logger);
 }
 

src/cli_plugin/remove/index.js

@@ -3,6 +3,7 @@ import remove from './remove';
 import Logger from '../lib/logger';
 import { parse } from './settings';
 import { getConfig } from '../../server/path';
+import logWarnings from '../lib/log_warnings';
 
 function processCommand(command, options) {
   let settings;
@@ -15,6 +16,7 @@ function processCommand(command, options) {
   }
 
   const logger = new Logger(settings);
+  logWarnings(settings, logger);
   remove(settings, logger);
 }
 

src/server/kbn_server.js

@@ -19,6 +19,7 @@ module.exports = class KbnServer {
       require('./config/setup'), // sets this.config, reads this.settings
       require('./http'), // sets this.server
       require('./logging'),
+      require('./warnings'),
       require('./status'),
 
       // writes pid file

src/server/warnings/index.js

@@ -0,0 +1,5 @@
+export default function (kbnServer, server, config) {
+  process.on('warning', (warning) => {
+    server.log(['warning', 'process'], warning);
+  });
+}