feat: improvement on log, debug and trace usage.

* Related to NODE-1814.

Author: helio-frota

OTEL.md

@@ -52,11 +52,9 @@ const exporter = new JaegerExporter({
 
 7. Deploy the example to OpenShift local
 
-* Uncomment the `require('./tracing.js');` on `app.js`
-
 ```
 npm install
-npm run openshift
+npm run openshift:enable:trace
 ```
 
 8. When you login on Jaeger UI you can see the result like this:

README.md

@@ -15,13 +15,12 @@ $ npm install
 $ npm start
 ```
 
-If you want debug information, you can set `DEBUG` environment variable and start the application:
+This will launch the application on port 8080.
 
-```
-$ DEBUG=* npm start
-```
+Other options:
 
-This will launch the application on port 8080.
+* `npm run dev` same as `npm start` but with pretty output log.
+* `npm run dev:debug` shows debug information.
 
 ### OpenShift Local
 
@@ -38,5 +37,4 @@ $ npm run openshift # Deploys the example app
 
 ### OpenTelemetry with OpenShift Distributed Tracing Platform
 
-This [link](./OTEL.md) contains instructions on how to install the 
-OpenShift Distributed Tracing Platform and enable tracing. 
+This [link](./OTEL.md) contains instructions on how to install the OpenShift Distributed Tracing Platform and enable tracing.

app.js

@@ -18,8 +18,6 @@
  *
  */
 
-// require('./tracing.js');
-
 const path = require('path');
 const express = require('express');
 const bodyParser = require('body-parser');

bin/www

@@ -24,15 +24,7 @@
  * Module dependencies.
  */
 
-const pinoDebug = require('pino-debug');
-const pino = require('pino');
-const logger = pino({
-  transport: {
-    target: 'pino-pretty'
-  }
-});
-pinoDebug(logger);
-
+const logger = require('../logger.js');
 const app = require('../app');
 const http = require('http');
 
@@ -93,10 +85,10 @@ function onError (error) {
   // handle specific listen errors with friendly messages
   switch (error.code) {
     case 'EACCES':
-      console.error(bind + ' requires elevated privileges');
+      logger.error(bind + ' requires elevated privileges');
       process.exit(1);
     case 'EADDRINUSE':
-      console.error(bind + ' is already in use');
+      logger.error(bind + ' is already in use');
       process.exit(1);
     default:
       throw error;

logger.js

@@ -0,0 +1,3 @@
+const pino = require('pino');
+
+module.exports = pino({});