azify-logger 1.0.54-test → 1.0.54-test-1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azify-logger",
-  "version": "1.0.54-test",
+  "version": "1.0.54-test-1",
   "description": "Azify Logger Client - Centralized logging for OpenSearch",
   "main": "index.js",
   "types": "index.d.ts",

package/register.js

@@ -10,7 +10,12 @@ try {
   }
   const ModuleForRequire = require('module')
   const nodeRequireOriginal = ModuleForRequire.prototype.require
-  const bunyan = require('bunyan')
+  let bunyan = null
+  try {
+    bunyan = require('bunyan')
+  } catch (_) {
+    /* bunyan é opcional; sem ele o patch Bunyan não aplica, mas HTTP/Pino/axios devem continuar */
+  }
   const createBunyanStream = require('./streams/bunyan')
   const { createHttpLoggerTransport } = require('./streams/httpQueue')
   const { getRequestContext } = require('./store')
@@ -146,7 +151,6 @@ try {
   const debug = process.env.AZIFY_LOGGER_DEBUG === '1'
   const httpVerbose = process.env.AZIFY_LOGGER_HTTP_VERBOSE === '1'
 
-  /** Quando o patch não consegue ler a URL dos args (undici/http), meta.url fica 'unknown', mas a mensagem ainda traz a URL. */
   function extractUrlFromReqResMessage(msgStr) {
     const s = String(msgStr)
     const m1 = s.match(/\[(?:REQUEST|RESPONSE)\]\s+\S+\s+(https?:\/\/\S+)/i)
@@ -156,6 +160,8 @@ try {
     return null
   }
 
+  const OPAQUE_NO_URL = 'https://opaque.invalid/azify-logger-no-url'
+
   function sendOutboundLog(level, message, meta) {
     try {
       const msgStr = String(message)
@@ -168,13 +174,10 @@ try {
           String(u).toLowerCase() === 'unknown'
         if (badUrl) {
           const fromMsg = extractUrlFromReqResMessage(msgStr)
-          if (fromMsg) {
-            meta = { ...meta, url: fromMsg }
-          }
+          meta = fromMsg ? { ...meta, url: fromMsg } : { ...meta, url: OPAQUE_NO_URL }
         }
       }
       if (isReqRes && meta && meta.url && isLoggerApiCall({ url: meta.url })) return
-      if (isReqRes && meta && (meta.url === 'unknown' || !String(meta.url || '').trim() || String(meta.url).toLowerCase() === 'unknown')) return
       const source = meta && meta.__source
       if (debug && source === 'pino-stdout') {
         if (debug) try { process.stderr.write('[azify-logger] SENDOUTBOUND_ENTER level=' + level + ' source=' + source + ' msg=' + msgStr.slice(0, 60) + '\n') } catch (_) {}
@@ -387,22 +390,24 @@ try {
     return false
   }
 
-  const originalCreate = bunyan.createLogger
-  bunyan.createLogger = function patchedCreateLogger(options) {
-    const logger = originalCreate.call(bunyan, options)
-    try {
-      const level = process.env.AZIFY_LOG_LEVEL || (options && options.level) || 'info'
-      const loggerUrl = process.env.AZIFY_LOGGER_URL
-      const serviceName = process.env.APP_NAME || (options && options.name)
-      const environment = process.env.NODE_ENV
-
-      logger.addStream({
-        level,
-        type: 'raw',
-        stream: createBunyanStream({ loggerUrl, serviceName, environment })
-      })
-    } catch (_) {}
-    return logger
+  if (bunyan && typeof bunyan.createLogger === 'function') {
+    const originalCreate = bunyan.createLogger
+    bunyan.createLogger = function patchedCreateLogger(options) {
+      const logger = originalCreate.call(bunyan, options)
+      try {
+        const level = process.env.AZIFY_LOG_LEVEL || (options && options.level) || 'info'
+        const loggerUrl = process.env.AZIFY_LOGGER_URL
+        const serviceName = process.env.APP_NAME || (options && options.name)
+        const environment = process.env.NODE_ENV
+
+        logger.addStream({
+          level,
+          type: 'raw',
+          stream: createBunyanStream({ loggerUrl, serviceName, environment })
+        })
+      } catch (_) {}
+      return logger
+    }
   }
 
   const getOtelTraceContext = () => {