bump version

bugfix: mem leak
update deps
2026-10-04 02:04:19 +00:00 · 2022-04-06 08:18:22 -04:00 · 2022-04-05 19:57:03 -04:00 · 2022-04-01 13:46:16 -04:00 · 2022-03-28 09:09:14 -04:00 · 2022-03-23 09:26:04 -04:00
5 changed files with 45 additions and 24 deletions
@@ -1,4 +1,4 @@
-FROM node:17.7.1-slim
+FROM node:17-slim
 WORKDIR /opt/app/
 COPY package.json ./
 RUN npm install
@@ -43,7 +43,8 @@ const initCdr = (srf, req) => {
    attempted_at: Date.now(),
    direction: 'outbound',
    host: srf.locals.sipAddress,
-    remote_host: uri.host
+    remote_host: uri.host,
+    trace_id: req.get('X-Trace-ID') || '00000000000000000000000000000000'
  };
 };

@@ -286,6 +287,7 @@ class CallSession extends Emitter {
              'all',
              '-X-MS-Teams-FQDN',
              '-X-MS-Teams-Tenant-FQDN',
+              '-X-Trace-ID',
              'X-CID',
              '-Allow',
              '-Session-Expires',
@@ -387,7 +389,7 @@ class CallSession extends Emitter {
              this.writeCdrs({...this.req.locals.cdr,
                terminated_at: Date.now(),
                termination_reason: 487 === status ? 'caller abandoned' : 'failed',
-                sip_status: status,
+                sip_status: status
              }).catch((err) => this.logger.error({err}, 'Error writing cdr for call failure'));
            }
          }
@@ -419,12 +421,14 @@ class CallSession extends Emitter {
    this.uas = uas;
    this.uac = uac;
    [uas, uac].forEach((dlg) => {
-      dlg.on('destroy', () => {
-        this.logger.info('call ended');
+      dlg.on('destroy', async() => {
+        const other = dlg.other;
        this.rtpEngineResource.destroy();
        this.activeCallIds.delete(this.req.get('Call-ID'));
        this.unsubscribeDTMF(this.logger, this.req.get('Call-ID'), this.rtpEngineOpts.uac.tag);
-        dlg.other.destroy();
+        try {
+          await other.destroy();
+        } catch (err) {}

        this.decrKey(this.callCountKey)
          .then((count) => {
@@ -445,6 +449,14 @@ class CallSession extends Emitter {
            duration: Math.floor((now - callStart) / 1000)
          }).catch((err) => this.logger.error({err}, 'Error writing cdr for completed call'));
        }
+        /* de-link the 2 Dialogs for GC */
+        dlg.removeAllListeners();
+        other.removeAllListeners();
+        dlg.other = null;
+        other.other = null;
+
+        this.logger.info(`call ended with normal termination, there are ${this.activeCallIds.size} active`);
+
        this.srf.endSession(this.req);
      });
    });
@@ -17,7 +17,12 @@ module.exports = (srf, logger, opts) => {
    req.locals = req.locals || {};
    const callId = req.get('Call-ID');
    req.locals.account_sid = req.get('X-Account-Sid');
-    req.locals.logger = logger.child({callId, account_sid: req.locals.account_sid});
+    const traceId = req.locals.trace_id = req.get('X-Trace-ID');
+    req.locals.logger = logger.child({
+      callId,
+      traceId,
+      account_sid:
+      req.locals.account_sid});

    if (!req.locals.account_sid) {
      logger.info('missing X-Account-Sid on outbound call');
@@ -15,7 +15,7 @@
        "@jambonz/realtimedb-helpers": "^0.4.24",
        "@jambonz/rtpengine-utils": "^0.3.1",
        "@jambonz/stats-collector": "^0.1.6",
-        "@jambonz/time-series": "^0.1.6",
+        "@jambonz/time-series": "^0.1.9",
        "cidr-matcher": "^2.1.1",
        "debug": "^4.3.3",
        "drachtio-fn-b2b-sugar": "^0.0.12",
@@ -1449,8 +1449,9 @@
      }
    },
    "node_modules/@jambonz/time-series": {
-      "version": "0.1.6",
-      "integrity": "sha512-EFO1ZFUazHLCzDYkAbpke0IfmPRogLkGg+KGWwz3SGpnuiltfohQ3TsJ5KG8Gp9mXqdpkyAioWSdaJxGDquthQ==",
+      "version": "0.1.9",
+      "resolved": "https://registry.npmjs.org/@jambonz/time-series/-/time-series-0.1.9.tgz",
+      "integrity": "sha512-FLD7mLGEMToG7s6LGWr/GtNp61753RBxyYuKSxCVI6G14bm6ydgcBdG85h8EGTVQlIp9dnwbn5ebSVsGb8a46w==",
      "dependencies": {
        "debug": "^4.3.1",
        "influx": "^5.8.0"
@@ -4093,8 +4094,9 @@
      }
    },
    "node_modules/minimist": {
-      "version": "1.2.5",
-      "integrity": "sha512-FM9nNUYrRBAELZQT3xeZQ7fmMOBg6nWNmJKTcgsJeaLstP/UODVpGsr5OhXhhXg6f+qtJ8uiZ+PUxkDWcgIXLw=="
+      "version": "1.2.6",
+      "resolved": "https://registry.npmjs.org/minimist/-/minimist-1.2.6.tgz",
+      "integrity": "sha512-Jsjnk4bw3YJqYzbdyBiNsPWHPfO++UGG749Cxs6peCu5Xg4nrena6OVxOYxrQTqww0Jmwt+Ref8rggumkTLz9Q=="
    },
    "node_modules/mkdirp": {
      "version": "0.5.5",
@@ -4187,9 +4189,9 @@
      }
    },
    "node_modules/node-forge": {
-      "version": "1.2.1",
-      "resolved": "https://registry.npmjs.org/node-forge/-/node-forge-1.2.1.tgz",
-      "integrity": "sha512-Fcvtbb+zBcZXbTTVwqGA5W+MKBj56UjVRevvchv5XrcyXbmNdesfZL37nlcWOfpgHhgmxApw3tQbTr4CqNmX4w==",
+      "version": "1.3.0",
+      "resolved": "https://registry.npmjs.org/node-forge/-/node-forge-1.3.0.tgz",
+      "integrity": "sha512-08ARB91bUi6zNKzVmaj3QO7cr397uiDT2nJ63cHjyNtCTWIgvS47j3eT0WfzUwS9+6Z5YshRaoasFkXCKrIYbA==",
      "engines": {
        "node": ">= 6.13.0"
      }
@@ -6874,8 +6876,9 @@
      }
    },
    "@jambonz/time-series": {
-      "version": "0.1.6",
-      "integrity": "sha512-EFO1ZFUazHLCzDYkAbpke0IfmPRogLkGg+KGWwz3SGpnuiltfohQ3TsJ5KG8Gp9mXqdpkyAioWSdaJxGDquthQ==",
+      "version": "0.1.9",
+      "resolved": "https://registry.npmjs.org/@jambonz/time-series/-/time-series-0.1.9.tgz",
+      "integrity": "sha512-FLD7mLGEMToG7s6LGWr/GtNp61753RBxyYuKSxCVI6G14bm6ydgcBdG85h8EGTVQlIp9dnwbn5ebSVsGb8a46w==",
      "requires": {
        "debug": "^4.3.1",
        "influx": "^5.8.0"
@@ -8849,8 +8852,9 @@
      }
    },
    "minimist": {
-      "version": "1.2.5",
-      "integrity": "sha512-FM9nNUYrRBAELZQT3xeZQ7fmMOBg6nWNmJKTcgsJeaLstP/UODVpGsr5OhXhhXg6f+qtJ8uiZ+PUxkDWcgIXLw=="
+      "version": "1.2.6",
+      "resolved": "https://registry.npmjs.org/minimist/-/minimist-1.2.6.tgz",
+      "integrity": "sha512-Jsjnk4bw3YJqYzbdyBiNsPWHPfO++UGG749Cxs6peCu5Xg4nrena6OVxOYxrQTqww0Jmwt+Ref8rggumkTLz9Q=="
    },
    "mkdirp": {
      "version": "0.5.5",
@@ -8922,9 +8926,9 @@
      }
    },
    "node-forge": {
-      "version": "1.2.1",
-      "resolved": "https://registry.npmjs.org/node-forge/-/node-forge-1.2.1.tgz",
-      "integrity": "sha512-Fcvtbb+zBcZXbTTVwqGA5W+MKBj56UjVRevvchv5XrcyXbmNdesfZL37nlcWOfpgHhgmxApw3tQbTr4CqNmX4w=="
+      "version": "1.3.0",
+      "resolved": "https://registry.npmjs.org/node-forge/-/node-forge-1.3.0.tgz",
+      "integrity": "sha512-08ARB91bUi6zNKzVmaj3QO7cr397uiDT2nJ63cHjyNtCTWIgvS47j3eT0WfzUwS9+6Z5YshRaoasFkXCKrIYbA=="
    },
    "node-noop": {
      "version": "0.0.1",
@@ -1,6 +1,6 @@
 {
  "name": "sbc-outbound",
-  "version": "v0.7.4",
+  "version": "v0.7.5",
  "main": "app.js",
  "engines": {
    "node": ">= 12.0.0"
@@ -33,7 +33,7 @@
    "@jambonz/realtimedb-helpers": "^0.4.24",
    "@jambonz/rtpengine-utils": "^0.3.1",
    "@jambonz/stats-collector": "^0.1.6",
-    "@jambonz/time-series": "^0.1.6",
+    "@jambonz/time-series": "^0.1.9",
    "cidr-matcher": "^2.1.1",
    "debug": "^4.3.3",
    "drachtio-fn-b2b-sugar": "^0.0.12",
Author	SHA1	Message	Date
Dave Horton	0c8967acdb	bump version	2022-04-06 08:18:22 -04:00
Dave Horton	1dfeed3ac8	bugfix: mem leak	2022-04-05 19:57:03 -04:00
Dave Horton	d10bda2926	update deps	2022-04-01 13:46:16 -04:00
Dave Horton	3938773738	mem leak fix	2022-03-28 09:09:14 -04:00
Dave Horton	f881002943	write otel trace_id to call history	2022-03-23 09:26:04 -04:00