@routr/connect 2.15.0 → 2.15.1

package/dist/handlers/register.js

@@ -71,6 +71,7 @@ const logger = (0, logger_1.getLogger)({ service: "connect", filePath: __filenam
 const jwtVerifier = (0, utils_1.getVerifierImpl)();
 const handleRegister = (apiClient, location) => {
     return (request, res) => __awaiter(void 0, void 0, void 0, function* () {
+        var _a;
         // Calculate and return challenge
         if (request.message.authorization) {
             const auth = Object.assign({}, request.message.authorization);
@@ -118,7 +119,7 @@ const handleRegister = (apiClient, location) => {
                 yield location.addRoute({
                     aor: payload.aor,
                     route: location_1.Helper.createRoute(request),
-                    maxContacts: payload.maxContacts || -1
+                    maxContacts: (_a = payload.maxContacts) !== null && _a !== void 0 ? _a : -1
                 });
                 res.sendRegisterOk(request);
             }

package/dist/handlers/request.js

@@ -76,7 +76,19 @@ const handleRequest = (location, apiClient) => (request, res) => __awaiter(void
     try {
         const req = common_1.Environment.ENFORCE_E164 ? enforceE164(request) : request;
         let route;
-        if (processor_1.Extensions.getHeaderValue(req, common_1.CommonTypes.ExtraHeader.EDGEPORT_REF)) {
+        // A CANCEL must follow the exact path its INVITE took (RFC 3261 §9.1) — it is
+        // never a new call to route. edgeport already forwards it correctly, straight
+        // from the INVITE's own client transaction (GRPCSipListener#sendCancel), which
+        // does not use the route this handler computes for CANCEL at all; it only needs
+        // a non-error response to reach that logic. Routing it fresh through
+        // peerToPSTN/agentToPSTN re-applies checks meant for a NEW call — chiefly
+        // peerToPSTN's required X-DOD-Number header, which Asterisk's CANCEL does not
+        // carry (only the original INVITE does) — and rejects it with a 400 that never
+        // reaches edgeport's already-correct CANCEL handling. The result: the CANCEL is
+        // silently dropped, the call keeps ringing, and if the callee answers a moment
+        // later they connect into a line the caller has already abandoned.
+        if (processor_1.Extensions.getHeaderValue(req, common_1.CommonTypes.ExtraHeader.EDGEPORT_REF) ||
+            req.method === common_1.CommonTypes.Method.CANCEL) {
             route = location_1.Helper.createRouteFromLastMessage(req);
         }
         else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@routr/connect",
-  "version": "2.15.0",
+  "version": "2.15.1",
   "description": "Default processor",
   "author": "Pedro Sanders <psanders@fonoster.com>",
   "homepage": "https://github.com/fonoster/routr#readme",
@@ -29,8 +29,8 @@
     "@opentelemetry/sdk-trace-node": "^1.0.4",
     "@opentelemetry/semantic-conventions": "^1.0.4",
     "@routr/common": "^2.15.0",
-    "@routr/location": "^2.15.0",
-    "@routr/processor": "^2.15.0",
+    "@routr/location": "^2.15.1",
+    "@routr/processor": "^2.15.1",
     "jsonwebtoken": "^9.0.3"
   },
   "devDependencies": {
@@ -49,5 +49,5 @@
   "bugs": {
     "url": "https://github.com/fonoster/routr/issues"
   },
-  "gitHead": "4cfe8529e9325b7a2f32aff615cf3db65c71cf29"
+  "gitHead": "2784f886cab933977feed3fe50edcaec0e6f7dac"
 }