Use pino instead of console.log (GoogleCloudPlatform#2263)

* Updated Loggers To Use 'pino' Instead Of 'console.log'

* Initialized pino

* Created a separate file of logger for paymentservice

* Resolving `header-check` Issue

* Moved `logger.js` inside paymentservice and resolved `header-check` issue

---------

Co-authored-by: Nim Jayawardena <[email protected]>

Author: DracFiendMG

src/currencyservice/server.js

@@ -14,11 +14,22 @@
  * limitations under the License.
  */
 
+const pino = require('pino');
+const logger = pino({
+  name: 'currencyservice-server',
+  messageKey: 'message',
+  formatters: {
+    level (logLevelString, logLevelNum) {
+      return { severity: logLevelString }
+    }
+  }
+});
+
 if(process.env.DISABLE_PROFILER) {
-  console.log("Profiler disabled.")
+  logger.info("Profiler disabled.")
 }
 else {
-  console.log("Profiler enabled.")
+  logger.info("Profiler enabled.")
   require('@google-cloud/profiler').start({
     serviceContext: {
       service: 'currencyservice',
@@ -37,7 +48,7 @@ registerInstrumentations({
 });
 
 if(process.env.ENABLE_TRACING == "1") {
-  console.log("Tracing enabled.")
+  logger.info("Tracing enabled.")
   const { NodeTracerProvider } = require('@opentelemetry/sdk-trace-node');
   const { SimpleSpanProcessor } = require('@opentelemetry/sdk-trace-base');
   const { OTLPTraceExporter } = require("@opentelemetry/exporter-otlp-grpc");
@@ -56,12 +67,11 @@ if(process.env.ENABLE_TRACING == "1") {
   provider.register();
 }
 else {
-  console.log("Tracing disabled.")
+  logger.info("Tracing disabled.")
 }
 
 const path = require('path');
 const grpc = require('@grpc/grpc-js');
-const pino = require('pino');
 const protoLoader = require('@grpc/proto-loader');
 
 const MAIN_PROTO_PATH = path.join(__dirname, './proto/demo.proto');
@@ -72,16 +82,6 @@ const PORT = process.env.PORT;
 const shopProto = _loadProto(MAIN_PROTO_PATH).hipstershop;
 const healthProto = _loadProto(HEALTH_PROTO_PATH).grpc.health.v1;
 
-const logger = pino({
-  name: 'currencyservice-server',
-  messageKey: 'message',
-  formatters: {
-    level (logLevelString, logLevelNum) {
-      return { severity: logLevelString }
-    }
-  }
-});
-
 /**
  * Helper function that loads a protobuf file.
  */

src/paymentservice/index.js

@@ -16,12 +16,13 @@
 
 'use strict';
 
+const logger = require('./logger')
 
 if(process.env.DISABLE_PROFILER) {
-  console.log("Profiler disabled.")
+  logger.info("Profiler disabled.")
 }
 else {
-  console.log("Profiler enabled.")
+  logger.info("Profiler enabled.")
   require('@google-cloud/profiler').start({
     serviceContext: {
       service: 'paymentservice',
@@ -32,7 +33,7 @@ else {
 
 
 if(process.env.ENABLE_TRACING == "1") {
-  console.log("Tracing enabled.")
+  logger.info("Tracing enabled.")
   const { NodeTracerProvider } = require('@opentelemetry/sdk-trace-node');
   const { SimpleSpanProcessor } = require('@opentelemetry/sdk-trace-base');
   const { GrpcInstrumentation } = require('@opentelemetry/instrumentation-grpc');
@@ -57,7 +58,7 @@ if(process.env.ENABLE_TRACING == "1") {
   });
 }
 else {
-  console.log("Tracing disabled.")
+  logger.info("Tracing disabled.")
 }
 
 

src/paymentservice/logger.js

@@ -0,0 +1,27 @@
+/*
+ * Copyright 2023 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+const pino = require('pino');
+
+module.exports = pino({
+  name: 'paymentservice-server',
+  messageKey: 'message',
+  formatters: {
+    level (logLevelString, logLevelNum) {
+      return { severity: logLevelString }
+    }
+  }
+});

src/paymentservice/server.js

@@ -14,20 +14,11 @@
 
 const path = require('path');
 const grpc = require('@grpc/grpc-js');
-const pino = require('pino');
 const protoLoader = require('@grpc/proto-loader');
 
 const charge = require('./charge');
 
-const logger = pino({
-  name: 'paymentservice-server',
-  messageKey: 'message',
-  formatters: {
-    level (logLevelString, logLevelNum) {
-      return { severity: logLevelString }
-    }
-  }
-});
+const logger = require('./logger')
 
 class HipsterShopServer {
   constructor(protoRoot, port = HipsterShopServer.PORT) {