azify-logger 1.0.44 → 1.0.46

package/streams/httpQueue.js

@@ -84,7 +84,7 @@ function resolveRedisConfig (overrides = {}) {
     (overrides.redis && overrides.redis.url) ||
     process.env.AZIFY_LOGGER_REDIS_URL ||
     DEFAULT_REDIS_URL
-  
+
   if (!redisUrl) {
     return null
   }
@@ -108,11 +108,13 @@ function createHttpLoggerTransport (loggerUrl, overrides) {
   if (!loggerUrl) {
     return noopTransport
   }
+  overrides = overrides || {}
 
   const options = resolveOptions(overrides)
   const redisConfig = resolveRedisConfig(overrides)
+  const hasRedisPassword = redisConfig && redisConfig.password != null && String(redisConfig.password).trim() !== ''
 
-  if (redisConfig) {
+  if (redisConfig && hasRedisPassword) {
     try {
       return createRedisStreamTransport(loggerUrl, options, redisConfig)
     } catch (err) {
@@ -124,9 +126,7 @@ function createHttpLoggerTransport (loggerUrl, overrides) {
   }
 
   const redisUrl = overrides.redisUrl || (overrides.redis && overrides.redis.url) || process.env.AZIFY_LOGGER_REDIS_URL || DEFAULT_REDIS_URL
-  const password = overrides.redisPassword ?? (overrides.redis && overrides.redis.password) ?? process.env.AZIFY_LOGGER_REDIS_PASSWORD
-  const pass = password != null && String(password).trim() !== '' ? String(password).trim() : undefined
-  if (redisUrl && !pass) {
+  if (redisUrl && !hasRedisPassword) {
     if (typeof global.__azifyLoggerRedisPasswordWarned === 'undefined') {
       global.__azifyLoggerRedisPasswordWarned = true
       process.stderr.write('[azify-logger] ⚠️  Redis requires a password. No password set. Using direct HTTP (no Redis). Set AZIFY_LOGGER_REDIS_PASSWORD to use Redis.\n')
@@ -250,7 +250,7 @@ function buildInlineTransport (loggerUrl, options) {
 
     flushTimer = setTimeout(() => {
       flushTimer = null
-      void flushQueue()
+      flushQueue().catch(() => {})
     }, delay)
 
     if (typeof flushTimer.unref === 'function') {
@@ -278,9 +278,15 @@ function buildInlineTransport (loggerUrl, options) {
     try {
       for (let i = 0; i < batch.length; i += 1) {
         const { payload, headers } = batch[i]
+        let body
+        try {
+          body = typeof payload === 'string' ? payload : JSON.stringify(payload)
+        } catch (_) {
+          continue
+        }
         try {
-          await axios.post(loggerUrl, payload, {
-            headers,
+          await axios.post(loggerUrl, body, {
+            headers: { ...headers, 'Content-Type': 'application/json' },
             timeout: options.timeout,
             httpAgent,
             httpsAgent,
@@ -298,11 +304,9 @@ function buildInlineTransport (loggerUrl, options) {
           if (remaining.length) {
             queue.unshift(...remaining)
           }
-
-          throw error
         }
       }
-    } catch (_) {
+    } catch (e) {
     } finally {
       flushing = false
       if (queue.length) {
@@ -311,6 +315,12 @@ function buildInlineTransport (loggerUrl, options) {
     }
   }
 
+  const isRequestOrResponsePayload = (p) => {
+    if (!p || typeof p !== 'object') return false
+    const msg = typeof p.message === 'string' ? p.message : ''
+    return msg.includes('[REQUEST]') || msg.includes('[RESPONSE]')
+  }
+
   const enqueue = (payload, headers = {}) => {
     if (queue.length >= options.maxQueueSize) {
       queue.shift()
@@ -319,7 +329,9 @@ function buildInlineTransport (loggerUrl, options) {
     queue.push({ payload, headers })
 
     if (queue.length >= options.batchSize) {
-      void flushQueue()
+      flushQueue().catch(() => {})
+    } else if (isRequestOrResponsePayload(payload)) {
+      scheduleFlush(0)
     } else {
       scheduleFlush()
     }

package/streams/pino.js

@@ -36,26 +36,49 @@ function createPinoStream(options = {}) {
 
       let traceId, spanId, parentSpanId
       try {
-        const { getRequestContext } = require('../store')
+        const { getRequestContext, getLastJobContext, toTraceIdHex } = require('../store')
         const ctx = getRequestContext()
         if (ctx && ctx.traceId) {
           traceId = ctx.traceId
           spanId = ctx.spanId
           parentSpanId = ctx.parentSpanId
         }
-        
-        if (!traceId && record.traceId) {
-          traceId = record.traceId
-          spanId = record.spanId
-          parentSpanId = record.parentSpanId
+
+        if (!traceId) {
+          const jobCtx = getLastJobContext()
+          if (jobCtx && jobCtx.traceId) {
+            traceId = jobCtx.traceId
+            spanId = jobCtx.spanId
+            parentSpanId = jobCtx.parentSpanId
+          }
+        }
+
+        if (!traceId && (record.traceId || (record.data && record.data.traceId))) {
+          traceId = record.traceId || (record.data && record.data.traceId)
+          spanId = record.spanId != null ? record.spanId : (record.data && record.data.spanId)
+          parentSpanId = record.parentSpanId != null ? record.parentSpanId : (record.data && record.data.parentSpanId)
         }
-        
+
         if (!traceId && record.req && record.req.traceId) {
           traceId = record.req.traceId
           spanId = record.req.spanId
           parentSpanId = record.req.parentSpanId
         }
-        
+
+        if (!traceId) {
+          const msgStr = record.msg || record.message || ''
+          if (typeof msgStr === 'string' && /traceId/i.test(msgStr)) {
+            const m = msgStr.match(/"traceId"\s*:\s*"([^"]+)"/)
+            if (m) {
+              traceId = m[1]
+              const s = msgStr.match(/"spanId"\s*:\s*"([^"]+)"/)
+              spanId = s ? s[1] : null
+              const p = msgStr.match(/"parentSpanId"\s*:\s*"([^"]*)"/)
+              parentSpanId = (p && p[1]) ? p[1] : null
+            }
+          }
+        }
+
         if (!traceId) {
           const span = trace.getSpan(context.active())
           const spanContext = span && span.spanContext()
@@ -65,6 +88,11 @@ function createPinoStream(options = {}) {
             parentSpanId = spanContext.parentSpanId
           }
         }
+
+        if (traceId && typeof traceId === 'string') {
+          traceId = toTraceIdHex(traceId)
+          if (spanId != null) spanId = String(spanId).slice(0, 16)
+        }
       } catch (_) {}
 
       const headers = {}
@@ -90,12 +118,19 @@ function createPinoStream(options = {}) {
         ...(parentSpanId && { parentSpanId })
       }
 
-      if (!shouldSample(level, 'logger')) {
+      const isErrorOrWarn = level === 'error' || level === 'fatal' || level === 'warn'
+      if (!isErrorOrWarn && !shouldSample(level, 'logger')) {
         return
       }
 
       const payload = { level, message: record.msg || record.message || 'log', meta }
-      transport.enqueue(payload, headers)
+      try {
+        transport.enqueue(payload, headers)
+      } catch (err) {
+        if (process.env.AZIFY_LOGGER_DEBUG === '1') {
+          process.stderr.write(`[azify-logger] pino stream enqueue error: ${err && err.message}\n`)
+        }
+      }
     }
   }
 }