@tellescope/sdk 1.256.13 → 1.256.15

package/lib/cjs/tests/api_tests/security/F-0192-automated-actions-process-gate.test.js

@@ -0,0 +1,324 @@
+"use strict";
+var __assign = (this && this.__assign) || function () {
+    __assign = Object.assign || function(t) {
+        for (var s, i = 1, n = arguments.length; i < n; i++) {
+            s = arguments[i];
+            for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p))
+                t[p] = s[p];
+        }
+        return t;
+    };
+    return __assign.apply(this, arguments);
+};
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+var __generator = (this && this.__generator) || function (thisArg, body) {
+    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g;
+    return g = { next: verb(0), "throw": verb(1), "return": verb(2) }, typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
+    function verb(n) { return function (v) { return step([n, v]); }; }
+    function step(op) {
+        if (f) throw new TypeError("Generator is already executing.");
+        while (g && (g = 0, op[0] && (_ = 0)), _) try {
+            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
+            if (y = 0, t) op = [op[0] & 2, t.value];
+            switch (op[0]) {
+                case 0: case 1: t = op; break;
+                case 4: _.label++; return { value: op[1], done: false };
+                case 5: _.label++; y = op[1]; op = [0]; continue;
+                case 7: op = _.ops.pop(); _.trys.pop(); continue;
+                default:
+                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
+                    if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
+                    if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
+                    if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
+                    if (t[2]) _.ops.pop();
+                    _.trys.pop(); continue;
+            }
+            op = body.call(thisArg, _);
+        } catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
+        if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
+    }
+};
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.automated_actions_process_gate_tests = void 0;
+require('source-map-support').install();
+var axios_1 = __importDefault(require("axios"));
+var sdk_1 = require("../../../sdk");
+var testing_1 = require("@tellescope/testing");
+var constants_1 = require("@tellescope/constants");
+var setup_1 = require("../../setup");
+var host = process.env.API_URL || 'http://localhost:8080';
+var RAND = function () { return Math.random().toString(36).slice(2, 10); };
+var ALL = 'All';
+var FULL_ACCESS = { create: ALL, read: ALL, update: ALL, delete: ALL };
+var PROCESS_PATH = '/v1/automated-actions/process';
+/**
+ * Regression test for F-0192
+ * (security-audit/findings/F-0192-automated-actions-process-fire-replay.md).
+ *
+ * `automated_actions.process` (api.ts:3902) is the WORKER's action runner — its schema description
+ * reads "Used by worker for new action types". It was registered with only
+ * `{ requiresSubscription: true }`, so the framework gated it on the ordinary
+ * `automated_actions.update` verb, which the default provider role holds ('Assigned'). The handler
+ * then dispatched `automatedAction.action` through a 14-branch if-chain reaching real vendors:
+ * chargebeeChargeCardOnFile (card charge), belugaAutoRx / belugaTriggerRefill / mdiTriggerRefill
+ * (prescriptions and refills), createScriptSureDraft (medication draft), healthiePushForms /
+ * metriportPushFormResponse (PHI export). It checked neither the row's `status` nor whether it had
+ * already run, so any row was replayable: MDI advances to the next scheduled refill each call,
+ * Chargebee charges again. And there was no storeUserLog, so the action was attributed to the
+ * worker rather than the caller.
+ *
+ * The fix has three parts:
+ *   1. `adminOnly: true` on the api.ts registration (routing.ts:944 reads the flag from that
+ *      options object — a schema-only flag is inert, which is the separate finding F-0191).
+ *   2. A single-fire claim for human callers: an atomic findOneAndUpdate with `status: 'active'` in
+ *      the filter, stamping `finished` + `processedAt` before the vendor dispatch. The worker
+ *      (`wasAutomated`) is exempt because its own claim in
+ *      worker/modules/database.ts:114 is already atomic and it needs the row to stay 'active'
+ *      across the call so a retryable failure (worker/modules/automations.ts:2053) can be
+ *      re-claimed after the lock times out.
+ *   3. storeUserLog on the human path, recording the caller and the automatedActionId.
+ *
+ * The seeded row uses `setEnduserStatus`, which is NOT one of the API's if-chain branches, so it
+ * falls through to `{ success: false, unimplemented: true }` (api.ts) — no vendor call, no Bedrock,
+ * no PHI export. `processAfter` is set an hour out so the live worker cannot race the row (the same
+ * trick mdi_trigger_refill / beluga_trigger_refill use).
+ *
+ * RED (pre-fix): A1 fails (a non-admin holding only `automated_actions.update` reaches the handler)
+ * and C1 fails (the second call on the same row executes again instead of being rejected).
+ * B1 (admin positive control) passes pre-fix and must keep passing.
+ * GREEN (post-fix): all five checks pass.
+ */
+var automated_actions_process_gate_tests = function (_a) {
+    var sdk = _a.sdk;
+    return __awaiter(void 0, void 0, void 0, function () {
+        var checks, record, post, roleId, attackerId, enduserId, automatedActionIds, role, rbap, attacker, attackerToken, adminToken, attackerSDK, enduser, seedAction, attackerRow, attackerAttempt, baselineRow, baseline, badProbe, attackerRowAfter, replay, baselineRowAfter, _i, checks_1, c, _b, checks_2, c, _c, automatedActionIds_1, id, _d, _e, _f, _g;
+        var _h, _j;
+        return __generator(this, function (_k) {
+            switch (_k.label) {
+                case 0:
+                    (0, testing_1.log_header)("F-0192: automated_actions.process must be admin-only and single-fire");
+                    checks = [];
+                    record = function (label, pass, detail) { return checks.push({ label: label, pass: pass, detail: detail }); };
+                    post = function (authToken, automatedActionId) { return __awaiter(void 0, void 0, void 0, function () {
+                        var r;
+                        return __generator(this, function (_a) {
+                            switch (_a.label) {
+                                case 0: return [4 /*yield*/, axios_1.default.request({
+                                        method: 'post',
+                                        url: "".concat(host).concat(PROCESS_PATH),
+                                        data: { automatedActionId: automatedActionId },
+                                        headers: { Authorization: "Bearer ".concat(authToken) },
+                                        validateStatus: function () { return true; },
+                                    })];
+                                case 1:
+                                    r = _a.sent();
+                                    return [2 /*return*/, { status: r.status, body: r.data }];
+                            }
+                        });
+                    }); };
+                    roleId = '';
+                    attackerId = '';
+                    enduserId = '';
+                    automatedActionIds = [];
+                    _k.label = 1;
+                case 1:
+                    _k.trys.push([1, , 15, 34]);
+                    role = "f0192-automations-".concat(RAND());
+                    return [4 /*yield*/, sdk.api.role_based_access_permissions.createOne({
+                            role: role,
+                            permissions: __assign(__assign({}, constants_1.PROVIDER_PERMISSIONS), { automated_actions: FULL_ACCESS }),
+                        })];
+                case 2:
+                    rbap = _k.sent();
+                    roleId = rbap.id;
+                    return [4 /*yield*/, sdk.api.users.createOne({
+                            email: "f0192-attacker-".concat(RAND(), "@tellescope.example"),
+                            fname: 'F0192', lname: 'Staff',
+                            verifiedEmail: true,
+                            notificationEmailsDisabled: true,
+                            roles: [role],
+                        })];
+                case 3:
+                    attacker = _k.sent();
+                    attackerId = attacker.id;
+                    return [4 /*yield*/, sdk.api.users.generate_auth_token({ id: attackerId })];
+                case 4:
+                    attackerToken = (_k.sent()).authToken;
+                    return [4 /*yield*/, sdk.api.users.generate_auth_token({ id: sdk.userInfo.id })];
+                case 5:
+                    adminToken = (_k.sent()).authToken;
+                    attackerSDK = new sdk_1.Session({ host: host, authToken: attackerToken });
+                    return [4 /*yield*/, attackerSDK.refresh_session()];
+                case 6:
+                    _k.sent();
+                    record('setup: attacker session is a non-admin holding automated_actions.update', attackerSDK.userInfo.id === attackerId && !((_h = attackerSDK.userInfo.roles) !== null && _h !== void 0 ? _h : []).includes('Admin'), "id=".concat(attackerSDK.userInfo.id, " roles=").concat(JSON.stringify(attackerSDK.userInfo.roles)));
+                    return [4 /*yield*/, sdk.api.endusers.createOne({ fname: 'F0192', lname: 'Patient' })];
+                case 7:
+                    enduser = _k.sent();
+                    enduserId = enduser.id;
+                    seedAction = function () { return __awaiter(void 0, void 0, void 0, function () {
+                        var a;
+                        return __generator(this, function (_a) {
+                            switch (_a.label) {
+                                case 0: return [4 /*yield*/, sdk.api.automated_actions.createOne({
+                                        enduserId: enduserId,
+                                        journeyId: constants_1.PLACEHOLDER_ID,
+                                        automationStepId: constants_1.PLACEHOLDER_ID,
+                                        event: { type: 'onJourneyStart', info: {} },
+                                        action: { type: 'setEnduserStatus', info: { status: 'Working' } },
+                                        status: 'active',
+                                        processAfter: Date.now() + 60 * 60 * 1000,
+                                    })];
+                                case 1:
+                                    a = _a.sent();
+                                    automatedActionIds.push(a.id);
+                                    return [2 /*return*/, a];
+                            }
+                        });
+                    }); };
+                    return [4 /*yield*/, seedAction()];
+                case 8:
+                    attackerRow = _k.sent();
+                    return [4 /*yield*/, post(attackerToken, attackerRow.id)
+                        // Baseline on the same path with an admin token, so a blanket 401 (expired subscription,
+                        // bad route, broken fixture) cannot masquerade as a working gate.
+                    ];
+                case 9:
+                    attackerAttempt = _k.sent();
+                    return [4 /*yield*/, seedAction()];
+                case 10:
+                    baselineRow = _k.sent();
+                    return [4 /*yield*/, post(adminToken, baselineRow.id)];
+                case 11:
+                    baseline = _k.sent();
+                    badProbe = baseline.status === 401 || baseline.status === 403;
+                    record('A1: a non-admin holding automated_actions.update cannot call process', !badProbe && (attackerAttempt.status === 401 || attackerAttempt.status === 403), "attacker=".concat(attackerAttempt.status, " ").concat(JSON.stringify(attackerAttempt.body), " baseline(admin)=").concat(baseline.status)
+                        + (badProbe ? ' BAD PROBE: admin baseline was also rejected' : ''));
+                    return [4 /*yield*/, sdk.api.automated_actions.getOne(attackerRow.id)];
+                case 12:
+                    attackerRowAfter = _k.sent();
+                    record('A2: a rejected non-admin call leaves the row unfired (status still active)', attackerRowAfter.status === 'active', "status=".concat(attackerRowAfter.status, " (expected active)"));
+                    /* ===== B: positive control — an admin can still call it ===== */
+                    record('B1: an admin can still process a pending row', baseline.status === 200 && ((_j = baseline.body) === null || _j === void 0 ? void 0 : _j.unimplemented) === true, "status=".concat(baseline.status, " body=").concat(JSON.stringify(baseline.body)));
+                    return [4 /*yield*/, post(adminToken, baselineRow.id)];
+                case 13:
+                    replay = _k.sent();
+                    record('C1: processing the same row twice is rejected (no replay)', replay.status === 409, "second call status=".concat(replay.status, " body=").concat(JSON.stringify(replay.body), " (expected 409)"));
+                    return [4 /*yield*/, sdk.api.automated_actions.getOne(baselineRow.id)];
+                case 14:
+                    baselineRowAfter = _k.sent();
+                    record('C2: a processed row is marked finished so neither a replay nor the worker re-runs it', baselineRowAfter.status === 'finished', "status=".concat(baselineRowAfter.status, " (expected finished)"));
+                    /* ===== report: print full table first, then assert (harness exits on first failure) ===== */
+                    console.log('\n  F-0192 check summary:');
+                    for (_i = 0, checks_1 = checks; _i < checks_1.length; _i++) {
+                        c = checks_1[_i];
+                        console.log("    ".concat(c.pass ? 'PASS' : 'FAIL', "  ").concat(c.label, "  \u2014  ").concat(c.detail));
+                    }
+                    console.log('');
+                    for (_b = 0, checks_2 = checks; _b < checks_2.length; _b++) {
+                        c = checks_2[_b];
+                        (0, testing_1.assert)(c.pass, "".concat(c.label, " \u2014 ").concat(c.detail), c.label);
+                    }
+                    return [3 /*break*/, 34];
+                case 15:
+                    _c = 0, automatedActionIds_1 = automatedActionIds;
+                    _k.label = 16;
+                case 16:
+                    if (!(_c < automatedActionIds_1.length)) return [3 /*break*/, 21];
+                    id = automatedActionIds_1[_c];
+                    _k.label = 17;
+                case 17:
+                    _k.trys.push([17, 19, , 20]);
+                    return [4 /*yield*/, sdk.api.automated_actions.deleteOne(id)];
+                case 18:
+                    _k.sent();
+                    return [3 /*break*/, 20];
+                case 19:
+                    _d = _k.sent();
+                    return [3 /*break*/, 20];
+                case 20:
+                    _c++;
+                    return [3 /*break*/, 16];
+                case 21:
+                    if (!enduserId) return [3 /*break*/, 25];
+                    _k.label = 22;
+                case 22:
+                    _k.trys.push([22, 24, , 25]);
+                    return [4 /*yield*/, sdk.api.endusers.deleteOne(enduserId)];
+                case 23:
+                    _k.sent();
+                    return [3 /*break*/, 25];
+                case 24:
+                    _e = _k.sent();
+                    return [3 /*break*/, 25];
+                case 25:
+                    if (!attackerId) return [3 /*break*/, 29];
+                    _k.label = 26;
+                case 26:
+                    _k.trys.push([26, 28, , 29]);
+                    return [4 /*yield*/, sdk.api.users.deleteOne(attackerId)];
+                case 27:
+                    _k.sent();
+                    return [3 /*break*/, 29];
+                case 28:
+                    _f = _k.sent();
+                    return [3 /*break*/, 29];
+                case 29:
+                    if (!roleId) return [3 /*break*/, 33];
+                    _k.label = 30;
+                case 30:
+                    _k.trys.push([30, 32, , 33]);
+                    return [4 /*yield*/, sdk.api.role_based_access_permissions.deleteOne(roleId)];
+                case 31:
+                    _k.sent();
+                    return [3 /*break*/, 33];
+                case 32:
+                    _g = _k.sent();
+                    return [3 /*break*/, 33];
+                case 33: return [7 /*endfinally*/];
+                case 34: return [2 /*return*/];
+            }
+        });
+    });
+};
+exports.automated_actions_process_gate_tests = automated_actions_process_gate_tests;
+// Allow running this test file independently
+if (require.main === module) {
+    console.log("\uD83C\uDF10 Using API URL: ".concat(host));
+    var sdk_2 = new sdk_1.Session({ host: host });
+    var sdkNonAdmin_1 = new sdk_1.Session({ host: host });
+    var runTests = function () { return __awaiter(void 0, void 0, void 0, function () {
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0: return [4 /*yield*/, (0, setup_1.setup_tests)(sdk_2, sdkNonAdmin_1)];
+                case 1:
+                    _a.sent();
+                    return [4 /*yield*/, (0, exports.automated_actions_process_gate_tests)({ sdk: sdk_2, sdkNonAdmin: sdkNonAdmin_1 })];
+                case 2:
+                    _a.sent();
+                    return [2 /*return*/];
+            }
+        });
+    }); };
+    runTests()
+        .then(function () {
+        console.log("✅ F-0192 automated_actions.process gate test suite completed successfully");
+        process.exit(0);
+    })
+        .catch(function (error) {
+        console.error("❌ F-0192 automated_actions.process gate test suite failed:", error);
+        process.exit(1);
+    });
+}
+//# sourceMappingURL=F-0192-automated-actions-process-gate.test.js.map

package/lib/cjs/tests/api_tests/security/F-0192-automated-actions-process-gate.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"F-0192-automated-actions-process-gate.test.js","sourceRoot":"","sources":["../../../../../src/tests/api_tests/security/F-0192-automated-actions-process-gate.test.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA,OAAO,CAAC,oBAAoB,CAAC,CAAC,OAAO,EAAE,CAAC;AAExC,gDAAyB;AAEzB,oCAAsC;AACtC,+CAG4B;AAC5B,mDAA4E;AAC5E,qCAAyC;AAEzC,IAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,OAAO,IAAI,uBAAgC,CAAA;AAEpE,IAAM,IAAI,GAAG,cAAM,OAAA,IAAI,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,EAAvC,CAAuC,CAAA;AAE1D,IAAM,GAAG,GAAG,KAAc,CAAA;AAC1B,IAAM,WAAW,GAAG,EAAE,MAAM,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,EAAE,CAAA;AAExE,IAAM,YAAY,GAAG,+BAA+B,CAAA;AAWpD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAoCG;AACI,IAAM,oCAAoC,GAAG,UAAO,EAAgD;QAA9C,GAAG,SAAA;;;;;;;oBAC9D,IAAA,oBAAU,EAAC,sEAAsE,CAAC,CAAA;oBAE5E,MAAM,GAAY,EAAE,CAAA;oBACpB,MAAM,GAAG,UAAC,KAAa,EAAE,IAAa,EAAE,MAAc,IAAK,OAAA,MAAM,CAAC,IAAI,CAAC,EAAE,KAAK,OAAA,EAAE,IAAI,MAAA,EAAE,MAAM,QAAA,EAAE,CAAC,EAApC,CAAoC,CAAA;oBAI/F,IAAI,GAAG,UAAO,SAAiB,EAAE,iBAAyB;;;;wCACpD,qBAAM,eAAK,CAAC,OAAO,CAAC;wCAC5B,MAAM,EAAE,MAAM;wCACd,GAAG,EAAE,UAAG,IAAI,SAAG,YAAY,CAAE;wCAC7B,IAAI,EAAE,EAAE,iBAAiB,mBAAA,EAAE;wCAC3B,OAAO,EAAE,EAAE,aAAa,EAAE,iBAAU,SAAS,CAAE,EAAE;wCACjD,cAAc,EAAE,cAAM,OAAA,IAAI,EAAJ,CAAI;qCAC3B,CAAC,EAAA;;oCANI,CAAC,GAAG,SAMR;oCACF,sBAAO,EAAE,MAAM,EAAE,CAAC,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,EAAA;;;yBAC1C,CAAA;oBAEG,MAAM,GAAG,EAAE,CAAA;oBACX,UAAU,GAAG,EAAE,CAAA;oBACf,SAAS,GAAG,EAAE,CAAA;oBACZ,kBAAkB,GAAa,EAAE,CAAA;;;;oBAO/B,IAAI,GAAG,4BAAqB,IAAI,EAAE,CAAE,CAAA;oBAC7B,qBAAM,GAAG,CAAC,GAAG,CAAC,6BAA6B,CAAC,SAAS,CAAC;4BACjE,IAAI,MAAA;4BACJ,WAAW,wBAAO,gCAAoB,KAAE,iBAAiB,EAAE,WAAW,GAAE;yBAClE,CAAC,EAAA;;oBAHH,IAAI,GAAG,SAGJ;oBACT,MAAM,GAAG,IAAI,CAAC,EAAE,CAAA;oBAKC,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC;4BAC7C,KAAK,EAAE,yBAAkB,IAAI,EAAE,wBAAqB;4BACpD,KAAK,EAAE,OAAO,EAAE,KAAK,EAAE,OAAO;4BAC9B,aAAa,EAAE,IAAI;4BACnB,0BAA0B,EAAE,IAAI;4BAChC,KAAK,EAAE,CAAC,IAAI,CAAC;yBACP,CAAC,EAAA;;oBANH,QAAQ,GAAG,SAMR;oBACT,UAAU,GAAG,QAAQ,CAAC,EAAE,CAAA;oBAGD,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,mBAAmB,CAAC,EAAE,EAAE,EAAE,UAAU,EAAE,CAAC,EAAA;;oBAA5E,aAAa,GAAG,CAAC,SAA2D,CAAC,CAAC,SAAS;oBACzE,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,mBAAmB,CAAC,EAAE,EAAE,EAAE,GAAG,CAAC,QAAQ,CAAC,EAAE,EAAE,CAAC,EAAA;;oBAA9E,UAAU,GAAG,CAAC,SAAgE,CAAC,CAAC,SAAS;oBAEzF,WAAW,GAAG,IAAI,aAAO,CAAC,EAAE,IAAI,MAAA,EAAE,SAAS,EAAE,aAAa,EAAE,CAAC,CAAA;oBACnE,qBAAM,WAAW,CAAC,eAAe,EAAE,EAAA;;oBAAnC,SAAmC,CAAA;oBACnC,MAAM,CACJ,yEAAyE,EACzE,WAAW,CAAC,QAAQ,CAAC,EAAE,KAAK,UAAU,IAAI,CAAC,CAAC,MAAA,WAAW,CAAC,QAAQ,CAAC,KAAK,mCAAI,EAAE,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,EAC/F,aAAM,WAAW,CAAC,QAAQ,CAAC,EAAE,oBAAU,IAAI,CAAC,SAAS,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAE,CACpF,CAAA;oBAEe,qBAAM,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,KAAK,EAAE,SAAS,EAAE,CAAC,EAAA;;oBAAhF,OAAO,GAAG,SAAsE;oBACtF,SAAS,GAAG,OAAO,CAAC,EAAE,CAAA;oBAKhB,UAAU,GAAG;;;;wCACP,qBAAM,GAAG,CAAC,GAAG,CAAC,iBAAiB,CAAC,SAAS,CAAC;wCAClD,SAAS,WAAA;wCACT,SAAS,EAAE,0BAAc;wCACzB,gBAAgB,EAAE,0BAAc;wCAChC,KAAK,EAAE,EAAE,IAAI,EAAE,gBAAgB,EAAE,IAAI,EAAE,EAAE,EAAE;wCAC3C,MAAM,EAAE,EAAE,IAAI,EAAE,kBAAkB,EAAE,IAAI,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE;wCACjE,MAAM,EAAE,QAAQ;wCAChB,YAAY,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI;qCACnC,CAAC,EAAA;;oCARH,CAAC,GAAG,SAQD;oCACT,kBAAkB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAA;oCAC7B,sBAAO,CAAC,EAAA;;;yBACT,CAAA;oBAImB,qBAAM,UAAU,EAAE,EAAA;;oBAAhC,WAAW,GAAG,SAAkB;oBACd,qBAAM,IAAI,CAAC,aAAa,EAAE,WAAW,CAAC,EAAE,CAAC;wBAEjE,yFAAyF;wBACzF,kEAAkE;sBAHD;;oBAA3D,eAAe,GAAG,SAAyC;oBAI7C,qBAAM,UAAU,EAAE,EAAA;;oBAAhC,WAAW,GAAG,SAAkB;oBACrB,qBAAM,IAAI,CAAC,UAAU,EAAE,WAAW,CAAC,EAAE,CAAC,EAAA;;oBAAjD,QAAQ,GAAG,SAAsC;oBACjD,QAAQ,GAAG,QAAQ,CAAC,MAAM,KAAK,GAAG,IAAI,QAAQ,CAAC,MAAM,KAAK,GAAG,CAAA;oBAEnE,MAAM,CACJ,sEAAsE,EACtE,CAAC,QAAQ,IAAI,CAAC,eAAe,CAAC,MAAM,KAAK,GAAG,IAAI,eAAe,CAAC,MAAM,KAAK,GAAG,CAAC,EAC/E,mBAAY,eAAe,CAAC,MAAM,cAAI,IAAI,CAAC,SAAS,CAAC,eAAe,CAAC,IAAI,CAAC,8BAAoB,QAAQ,CAAC,MAAM,CAAE;0BAC3G,CAAC,QAAQ,CAAC,CAAC,CAAC,8CAA8C,CAAC,CAAC,CAAC,EAAE,CAAC,CACrE,CAAA;oBAGwB,qBAAM,GAAG,CAAC,GAAG,CAAC,iBAAiB,CAAC,MAAM,CAAC,WAAW,CAAC,EAAE,CAAC,EAAA;;oBAAzE,gBAAgB,GAAG,SAAsD;oBAC/E,MAAM,CACJ,4EAA4E,EAC5E,gBAAgB,CAAC,MAAM,KAAK,QAAQ,EACpC,iBAAU,gBAAgB,CAAC,MAAM,uBAAoB,CACtD,CAAA;oBAED,kEAAkE;oBAElE,MAAM,CACJ,8CAA8C,EAC9C,QAAQ,CAAC,MAAM,KAAK,GAAG,IAAI,CAAA,MAAA,QAAQ,CAAC,IAAI,0CAAE,aAAa,MAAK,IAAI,EAChE,iBAAU,QAAQ,CAAC,MAAM,mBAAS,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAE,CAClE,CAAA;oBAIc,qBAAM,IAAI,CAAC,UAAU,EAAE,WAAW,CAAC,EAAE,CAAC,EAAA;;oBAA/C,MAAM,GAAG,SAAsC;oBACrD,MAAM,CACJ,2DAA2D,EAC3D,MAAM,CAAC,MAAM,KAAK,GAAG,EACrB,6BAAsB,MAAM,CAAC,MAAM,mBAAS,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,oBAAiB,CACzF,CAAA;oBAEwB,qBAAM,GAAG,CAAC,GAAG,CAAC,iBAAiB,CAAC,MAAM,CAAC,WAAW,CAAC,EAAE,CAAC,EAAA;;oBAAzE,gBAAgB,GAAG,SAAsD;oBAC/E,MAAM,CACJ,sFAAsF,EACtF,gBAAgB,CAAC,MAAM,KAAK,UAAU,EACtC,iBAAU,gBAAgB,CAAC,MAAM,yBAAsB,CACxD,CAAA;oBAED,8FAA8F;oBAC9F,OAAO,CAAC,GAAG,CAAC,2BAA2B,CAAC,CAAA;oBACxC,WAAsB,EAAN,iBAAM,EAAN,oBAAM,EAAN,IAAM,EAAE;wBAAb,CAAC;wBACV,OAAO,CAAC,GAAG,CAAC,cAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,eAAK,CAAC,CAAC,KAAK,uBAAQ,CAAC,CAAC,MAAM,CAAE,CAAC,CAAA;qBAC3E;oBACD,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAA;oBAEf,WAAsB,EAAN,iBAAM,EAAN,oBAAM,EAAN,IAAM,EAAE;wBAAb,CAAC;wBACV,IAAA,gBAAM,EAAC,CAAC,CAAC,IAAI,EAAE,UAAG,CAAC,CAAC,KAAK,qBAAM,CAAC,CAAC,MAAM,CAAE,EAAE,CAAC,CAAC,KAAK,CAAC,CAAA;qBACpD;;;0BAIkC,EAAlB,yCAAkB;;;yBAAlB,CAAA,gCAAkB,CAAA;oBAAxB,EAAE;;;;oBAAgC,qBAAM,GAAG,CAAC,GAAG,CAAC,iBAAiB,CAAC,SAAS,CAAC,EAAE,CAAC,EAAA;;oBAA7C,SAA6C,CAAA;;;;;;oBAAzE,IAAkB,CAAA;;;yBAC/B,SAAS,EAAT,yBAAS;;;;oBAAU,qBAAM,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,SAAS,CAAC,SAAS,CAAC,EAAA;;oBAA3C,SAA2C,CAAA;;;;;;yBAC9D,UAAU,EAAV,yBAAU;;;;oBAAU,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,UAAU,CAAC,EAAA;;oBAAzC,SAAyC,CAAA;;;;;;yBAC7D,MAAM,EAAN,yBAAM;;;;oBAAU,qBAAM,GAAG,CAAC,GAAG,CAAC,6BAA6B,CAAC,SAAS,CAAC,MAAM,CAAC,EAAA;;oBAA7D,SAA6D,CAAA;;;;;;;;;;CAEpF,CAAA;AApJY,QAAA,oCAAoC,wCAoJhD;AAED,6CAA6C;AAC7C,IAAI,OAAO,CAAC,IAAI,KAAK,MAAM,EAAE;IAC3B,OAAO,CAAC,GAAG,CAAC,sCAAqB,IAAI,CAAE,CAAC,CAAA;IACxC,IAAM,KAAG,GAAG,IAAI,aAAO,CAAC,EAAE,IAAI,MAAA,EAAE,CAAC,CAAA;IACjC,IAAM,aAAW,GAAG,IAAI,aAAO,CAAC,EAAE,IAAI,MAAA,EAAE,CAAC,CAAA;IAEzC,IAAM,QAAQ,GAAG;;;wBACf,qBAAM,IAAA,mBAAW,EAAC,KAAG,EAAE,aAAW,CAAC,EAAA;;oBAAnC,SAAmC,CAAA;oBACnC,qBAAM,IAAA,4CAAoC,EAAC,EAAE,GAAG,OAAA,EAAE,WAAW,eAAA,EAAE,CAAC,EAAA;;oBAAhE,SAAgE,CAAA;;;;SACjE,CAAA;IAED,QAAQ,EAAE;SACP,IAAI,CAAC;QACJ,OAAO,CAAC,GAAG,CAAC,2EAA2E,CAAC,CAAA;QACxF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC,CAAC;SACD,KAAK,CAAC,UAAC,KAAK;QACX,OAAO,CAAC,KAAK,CAAC,4DAA4D,EAAE,KAAK,CAAC,CAAA;QAClF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC,CAAC,CAAA;CACL"}

package/lib/cjs/tests/api_tests/security/F-0196-add-to-journey-foreign-enduserid.test.d.ts

@@ -0,0 +1,43 @@
+import { Check } from "../../setup_isolated";
+/**
+ * Regression test for F-0196
+ * (security-audit/findings/F-0196-add-to-journey-plants-foreign-enduserid.md).
+ *
+ * `endusers.add_to_journey` inserted an `automated_actions` row for EACH caller-supplied `enduserId`
+ * with only mongoId-shape validation and no tenant-membership check. The preceding
+ * `DB.endusers.updateMany` is scoped but its result was discarded, and
+ * `DB.automated_actions.insertMany` is a direct write that bypasses the `ensureValidDependentKeys`
+ * foreign-key check the generic CRUD route gets. The resulting row is stamped
+ * `businessId: <caller's tenant>` while carrying `enduserId: <a victim in another tenant>`.
+ *
+ * The worker then dereferences that enduser with the raw driver and no `businessId`
+ * (worker/modules/automations.ts, the F-0074/F-0111/F-0159 family), so the victim's record loads
+ * under the attacker's automation: `pushFormsToPortal` copies the victim's PHI into the attacker's
+ * `form_responses` (readable by their staff) and `iterableCustomEvent` writes an attacker-authored
+ * note into the victim's chart. F-0196 is the reachability that made those three exploitable.
+ *
+ * There are TWO independent plant sites and a fix to only one is a half fix:
+ *   - inline branch, taken when `automationStepId` is supplied (api.ts add_to_journey handler)
+ *   - module branch, taken when it is not: `handle_automation_event`'s
+ *     `validEnduserIds = conditionsQuery ? <scoped find> : event.enduserIds` passes caller ids
+ *     straight through for a step with NO conditions.
+ * Both are covered below (A1/A2).
+ *
+ * Design notes:
+ *   - Two provisioned tenants, so the victim is a real foreign enduser and we can also assert their
+ *     record was not mutated. Both must come from provisioning: `add_to_journey` is
+ *     `requiresSubscription: true`, and a `create_and_join` org is born with an expired subscription.
+ *   - The steps use `setEnduserStatus` — no vendor, no Bedrock — and `startAt` is an hour out so a
+ *     running worker cannot process the rows before we read them.
+ *   - `throttle` is deliberately omitted: it is the only way add_to_journey's per-pair rate limit can
+ *     throw.
+ *   - Everything is observable through `automated_actions` reads, so the worker is never needed.
+ *
+ * RED (pre-fix): A1, A2 and B2 fail — the victim's id appears in the attacker tenant's
+ * automated_actions. GREEN (post-fix): the foreign id is silently dropped and the caller's own
+ * endusers still enroll normally.
+ */
+export declare const add_to_journey_foreign_enduserid_checks: () => Promise<Check[]>;
+/** Asserting wrapper, for standalone runs and for sequential use in the suite. */
+export declare const add_to_journey_foreign_enduserid_tests: () => Promise<void>;
+//# sourceMappingURL=F-0196-add-to-journey-foreign-enduserid.test.d.ts.map

package/lib/cjs/tests/api_tests/security/F-0196-add-to-journey-foreign-enduserid.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"F-0196-add-to-journey-foreign-enduserid.test.d.ts","sourceRoot":"","sources":["../../../../../src/tests/api_tests/security/F-0196-add-to-journey-foreign-enduserid.test.ts"],"names":[],"mappings":"AAEA,OAAO,EAEL,KAAK,EAEN,MAAM,sBAAsB,CAAA;AAS7B;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqCG;AACH,eAAO,MAAM,uCAAuC,QAAa,QAAQ,KAAK,EAAE,CAuI/E,CAAA;AAED,kFAAkF;AAClF,eAAO,MAAM,sCAAsC,qBAElD,CAAA"}