@platformatic/basic 2.75.0-alpha.0 → 2.75.0-alpha.1

package/config.d.ts

@@ -95,6 +95,7 @@ export interface PlatformaticStackable {
     };
     startTimeout?: number;
     restartOnError?: boolean | number;
+    exitOnUnhandledErrors?: boolean;
     gracefulShutdown?: {
       runtime: number | string;
       service: number | string;

package/lib/base.js

@@ -352,6 +352,7 @@ export class BaseStackable extends EventEmitter {
       isEntrypoint: this.isEntrypoint,
       runtimeBasePath: this.runtimeConfig?.basePath ?? null,
       wantsAbsoluteUrls: meta.composer?.wantsAbsoluteUrls ?? false,
+      exitOnUnhandledErrors: this.runtimeConfig.exitOnUnhandledErrors ?? true,
       /* c8 ignore next 2 - else */
       port: (this.isEntrypoint ? this.serverConfig?.port || 0 : undefined) ?? true,
       host: (this.isEntrypoint ? this.serverConfig?.hostname : undefined) ?? true,
@@ -416,12 +417,7 @@ export class BaseStackable extends EventEmitter {
         return
       }
 
-      await collectMetrics(
-        this.serviceId,
-        this.workerId,
-        metricsConfig,
-        this.metricsRegistry
-      )
+      await collectMetrics(this.serviceId, this.workerId, metricsConfig, this.metricsRegistry)
     }
   }
 
@@ -512,7 +508,7 @@ export class BaseStackable extends EventEmitter {
       help: 'Number of active resources keeping the event loop alive',
       registers: [registry]
     })
-    globalThis.platformatic.onActiveResourcesEventLoop = (val) => activeResourcesEventLoopMetric.set(val)
+    globalThis.platformatic.onActiveResourcesEventLoop = val => activeResourcesEventLoopMetric.set(val)
   }
 
   async #invalidateHttpCache (opts = {}) {

package/lib/worker/child-process.js

@@ -83,7 +83,7 @@ export class ChildProcess extends ITC {
         getMetrics: (...args) => {
           return this.#getMetrics(...args)
         },
-        close: (signal) => {
+        close: signal => {
           let handled = false
 
           try {
@@ -113,7 +113,11 @@ export class ChildProcess extends ITC {
 
     this.listen()
     this.#setupLogger()
-    this.#setupHandlers()
+
+    if (globalThis.platformatic.exitOnUnhandledErrors) {
+      this.#setupHandlers()
+    }
+
     this.#setupServer()
     this.#setupInterceptors()
 
@@ -289,7 +293,7 @@ export class ChildProcess extends ITC {
       help: 'Number of active resources keeping the event loop alive',
       registers: [registry]
     })
-    globalThis.platformatic.onActiveResourcesEventLoop = (val) => activeResourcesEventLoopMetric.set(val)
+    globalThis.platformatic.onActiveResourcesEventLoop = val => activeResourcesEventLoopMetric.set(val)
   }
 
   async #getMetrics ({ format } = {}) {
@@ -406,6 +410,14 @@ export class ChildProcess extends ITC {
 
     process.on('uncaughtException', handleUnhandled.bind(this, 'uncaught exception'))
     process.on('unhandledRejection', handleUnhandled.bind(this, 'unhandled rejection'))
+
+    process.on('newListener', event => {
+      if (event === 'uncaughtException' || event === 'unhandledRejection') {
+        this.#logger.warn(
+          `A listener has been added for the "process.${event}" event. This listener will be never triggered as Watt default behavior will kill the process before.\n To disable this behavior, set "exitOnUnhandledErrors" to false in the runtime config.`
+        )
+      }
+    })
   }
 
   #notifyConfig (config) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/basic",
-  "version": "2.75.0-alpha.0",
+  "version": "2.75.0-alpha.1",
   "description": "",
   "main": "index.js",
   "type": "module",
@@ -24,11 +24,11 @@
     "split2": "^4.2.0",
     "undici": "^7.0.0",
     "ws": "^8.18.0",
-    "@platformatic/metrics": "2.75.0-alpha.0",
-    "@platformatic/config": "2.75.0-alpha.0",
-    "@platformatic/utils": "2.75.0-alpha.0",
-    "@platformatic/telemetry": "2.75.0-alpha.0",
-    "@platformatic/itc": "2.75.0-alpha.0"
+    "@platformatic/metrics": "2.75.0-alpha.1",
+    "@platformatic/config": "2.75.0-alpha.1",
+    "@platformatic/telemetry": "2.75.0-alpha.1",
+    "@platformatic/utils": "2.75.0-alpha.1",
+    "@platformatic/itc": "2.75.0-alpha.1"
   },
   "devDependencies": {
     "borp": "^0.20.0",

package/schema.json

@@ -1,5 +1,5 @@
 {
-  "$id": "https://schemas.platformatic.dev/@platformatic/basic/2.75.0-alpha.0.json",
+  "$id": "https://schemas.platformatic.dev/@platformatic/basic/2.75.0-alpha.1.json",
   "$schema": "http://json-schema.org/draft-07/schema#",
   "title": "Platformatic Stackable",
   "type": "object",
@@ -327,6 +327,10 @@
             }
           ]
         },
+        "exitOnUnhandledErrors": {
+          "default": true,
+          "type": "boolean"
+        },
         "gracefulShutdown": {
           "type": "object",
           "properties": {

package/lib/worker/child-transport.js

@@ -1,59 +0,0 @@
-import { generateRequest, sanitize } from '@platformatic/itc'
-import { ensureLoggableError } from '@platformatic/utils'
-import { once } from 'node:events'
-import { platform } from 'node:os'
-import { workerData } from 'node:worker_threads'
-import build from 'pino-abstract-transport'
-import { WebSocket } from 'ws'
-import { getSocketPath } from './child-manager.js'
-
-/* c8 ignore next 5 */
-function logDirectError (message, error) {
-  process._rawDebug(`Logger thread for child process of service ${workerData.id} ${message}.`, {
-    error: ensureLoggableError(error)
-  })
-}
-
-/* c8 ignore next 4 */
-function handleUnhandled (type, error) {
-  logDirectError(`threw an ${type}`, error)
-  process.exit(6)
-}
-
-process.on('uncaughtException', handleUnhandled.bind(null, 'uncaught exception'))
-process.on('unhandledRejection', handleUnhandled.bind(null, 'unhandled rejection'))
-
-export default async function () {
-  try {
-    /* c8 ignore next */
-    const protocol = platform() === 'win32' ? 'ws+unix:' : 'ws+unix://'
-    const socket = new WebSocket(`${protocol}${getSocketPath(process.env.PLT_MANAGER_ID)}`)
-
-    await once(socket, 'open')
-
-    // Do not process responses but empty the socket inbound queue
-    socket.on('message', () => {})
-
-    /* c8 ignore next 3 */
-    socket.on('error', error => {
-      logDirectError('threw a socket error', error)
-    })
-
-    return build(
-      async function (source) {
-        for await (const obj of source) {
-          socket.send(JSON.stringify(sanitize(generateRequest('log', { logs: [obj] }))))
-        }
-      },
-      {
-        close (_, cb) {
-          socket.close()
-          cb()
-        }
-      }
-    )
-    /* c8 ignore next 3 */
-  } catch (error) {
-    logDirectError('threw a connection error', error)
-  }
-}