@tellescope/sdk 1.252.0 → 1.252.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/lib/cjs/tests/api_tests/calendar_canvas_coding_clear.test.d.ts +6 -0
- package/lib/cjs/tests/api_tests/calendar_canvas_coding_clear.test.d.ts.map +1 -0
- package/lib/cjs/tests/api_tests/calendar_canvas_coding_clear.test.js +139 -0
- package/lib/cjs/tests/api_tests/calendar_canvas_coding_clear.test.js.map +1 -0
- package/lib/cjs/tests/api_tests/integrations_redacted.test.d.ts.map +1 -1
- package/lib/cjs/tests/api_tests/integrations_redacted.test.js +30 -20
- package/lib/cjs/tests/api_tests/integrations_redacted.test.js.map +1 -1
- package/lib/cjs/tests/api_tests/mdi_webhooks.test.d.ts +22 -0
- package/lib/cjs/tests/api_tests/mdi_webhooks.test.d.ts.map +1 -0
- package/lib/cjs/tests/api_tests/mdi_webhooks.test.js +162 -0
- package/lib/cjs/tests/api_tests/mdi_webhooks.test.js.map +1 -0
- package/lib/cjs/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.d.ts +32 -0
- package/lib/cjs/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.d.ts.map +1 -0
- package/lib/cjs/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.js +237 -0
- package/lib/cjs/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.js.map +1 -0
- package/lib/cjs/tests/api_tests/security/F-0076-self-admin-role-assignment.test.d.ts +38 -0
- package/lib/cjs/tests/api_tests/security/F-0076-self-admin-role-assignment.test.d.ts.map +1 -0
- package/lib/cjs/tests/api_tests/security/F-0076-self-admin-role-assignment.test.js +222 -0
- package/lib/cjs/tests/api_tests/security/F-0076-self-admin-role-assignment.test.js.map +1 -0
- package/lib/cjs/tests/api_tests/user_portal_settings.test.d.ts +6 -0
- package/lib/cjs/tests/api_tests/user_portal_settings.test.d.ts.map +1 -0
- package/lib/cjs/tests/api_tests/user_portal_settings.test.js +301 -0
- package/lib/cjs/tests/api_tests/user_portal_settings.test.js.map +1 -0
- package/lib/cjs/tests/tests.d.ts.map +1 -1
- package/lib/cjs/tests/tests.js +154 -142
- package/lib/cjs/tests/tests.js.map +1 -1
- package/lib/esm/tests/api_tests/calendar_canvas_coding_clear.test.d.ts +6 -0
- package/lib/esm/tests/api_tests/calendar_canvas_coding_clear.test.d.ts.map +1 -0
- package/lib/esm/tests/api_tests/calendar_canvas_coding_clear.test.js +135 -0
- package/lib/esm/tests/api_tests/calendar_canvas_coding_clear.test.js.map +1 -0
- package/lib/esm/tests/api_tests/integrations_redacted.test.d.ts.map +1 -1
- package/lib/esm/tests/api_tests/integrations_redacted.test.js +30 -20
- package/lib/esm/tests/api_tests/integrations_redacted.test.js.map +1 -1
- package/lib/esm/tests/api_tests/mdi_webhooks.test.d.ts +22 -0
- package/lib/esm/tests/api_tests/mdi_webhooks.test.d.ts.map +1 -0
- package/lib/esm/tests/api_tests/mdi_webhooks.test.js +158 -0
- package/lib/esm/tests/api_tests/mdi_webhooks.test.js.map +1 -0
- package/lib/esm/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.d.ts +32 -0
- package/lib/esm/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.d.ts.map +1 -0
- package/lib/esm/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.js +233 -0
- package/lib/esm/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.js.map +1 -0
- package/lib/esm/tests/api_tests/security/F-0076-self-admin-role-assignment.test.d.ts +38 -0
- package/lib/esm/tests/api_tests/security/F-0076-self-admin-role-assignment.test.d.ts.map +1 -0
- package/lib/esm/tests/api_tests/security/F-0076-self-admin-role-assignment.test.js +218 -0
- package/lib/esm/tests/api_tests/security/F-0076-self-admin-role-assignment.test.js.map +1 -0
- package/lib/esm/tests/api_tests/user_portal_settings.test.d.ts +6 -0
- package/lib/esm/tests/api_tests/user_portal_settings.test.d.ts.map +1 -0
- package/lib/esm/tests/api_tests/user_portal_settings.test.js +297 -0
- package/lib/esm/tests/api_tests/user_portal_settings.test.js.map +1 -0
- package/lib/esm/tests/tests.d.ts.map +1 -1
- package/lib/esm/tests/tests.js +154 -142
- package/lib/esm/tests/tests.js.map +1 -1
- package/lib/tsconfig.tsbuildinfo +1 -1
- package/package.json +10 -10
- package/src/tests/api_tests/integrations_redacted.test.ts +8 -0
- package/src/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.ts +161 -0
- package/src/tests/api_tests/security/F-0076-self-admin-role-assignment.test.ts +165 -0
- package/src/tests/api_tests/user_portal_settings.test.ts +217 -0
- package/src/tests/tests.ts +6 -0
- package/test_generated.pdf +0 -0
|
@@ -0,0 +1,237 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
var __assign = (this && this.__assign) || function () {
|
|
3
|
+
__assign = Object.assign || function(t) {
|
|
4
|
+
for (var s, i = 1, n = arguments.length; i < n; i++) {
|
|
5
|
+
s = arguments[i];
|
|
6
|
+
for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p))
|
|
7
|
+
t[p] = s[p];
|
|
8
|
+
}
|
|
9
|
+
return t;
|
|
10
|
+
};
|
|
11
|
+
return __assign.apply(this, arguments);
|
|
12
|
+
};
|
|
13
|
+
var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
|
|
14
|
+
function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
|
|
15
|
+
return new (P || (P = Promise))(function (resolve, reject) {
|
|
16
|
+
function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
|
|
17
|
+
function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
|
|
18
|
+
function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
|
|
19
|
+
step((generator = generator.apply(thisArg, _arguments || [])).next());
|
|
20
|
+
});
|
|
21
|
+
};
|
|
22
|
+
var __generator = (this && this.__generator) || function (thisArg, body) {
|
|
23
|
+
var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g;
|
|
24
|
+
return g = { next: verb(0), "throw": verb(1), "return": verb(2) }, typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
|
|
25
|
+
function verb(n) { return function (v) { return step([n, v]); }; }
|
|
26
|
+
function step(op) {
|
|
27
|
+
if (f) throw new TypeError("Generator is already executing.");
|
|
28
|
+
while (g && (g = 0, op[0] && (_ = 0)), _) try {
|
|
29
|
+
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;
|
|
30
|
+
if (y = 0, t) op = [op[0] & 2, t.value];
|
|
31
|
+
switch (op[0]) {
|
|
32
|
+
case 0: case 1: t = op; break;
|
|
33
|
+
case 4: _.label++; return { value: op[1], done: false };
|
|
34
|
+
case 5: _.label++; y = op[1]; op = [0]; continue;
|
|
35
|
+
case 7: op = _.ops.pop(); _.trys.pop(); continue;
|
|
36
|
+
default:
|
|
37
|
+
if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
|
|
38
|
+
if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
|
|
39
|
+
if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
|
|
40
|
+
if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
|
|
41
|
+
if (t[2]) _.ops.pop();
|
|
42
|
+
_.trys.pop(); continue;
|
|
43
|
+
}
|
|
44
|
+
op = body.call(thisArg, _);
|
|
45
|
+
} catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
|
|
46
|
+
if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
|
|
47
|
+
}
|
|
48
|
+
};
|
|
49
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
50
|
+
exports.cascade_role_rename_cross_tenant_tests = void 0;
|
|
51
|
+
require('source-map-support').install();
|
|
52
|
+
var sdk_1 = require("../../../sdk");
|
|
53
|
+
var testing_1 = require("@tellescope/testing");
|
|
54
|
+
var setup_1 = require("../../setup");
|
|
55
|
+
var constants_1 = require("@tellescope/constants");
|
|
56
|
+
var host = process.env.API_URL || 'http://localhost:8080';
|
|
57
|
+
// Separate tenant (different businessId), reusing the same hardcoded apiKey that
|
|
58
|
+
// multi_tenant_tests relies on in tests.ts.
|
|
59
|
+
var OTHER_TENANT_API_KEY = "ba745e25162bb95a795c5fa1af70df188d93c4d3aac9c48b34a5c8c9dd7b80f7";
|
|
60
|
+
/**
|
|
61
|
+
* Tenant-boundary guard for cascade_role_rename (relates to security-audit finding F-0053,
|
|
62
|
+
* which was investigated and closed as a FALSE POSITIVE — see that file for the code trace).
|
|
63
|
+
*
|
|
64
|
+
* The `cascade_role_rename` side-effect handler
|
|
65
|
+
* ([event_handlers_v2/role_based_access_permissions.ts](packages/private/api/api/v1/event_handlers_v2/role_based_access_permissions.ts))
|
|
66
|
+
* runs when a `role_based_access_permissions` doc's `role` field changes. It finds every user
|
|
67
|
+
* with the old role name (via `DBUnrestricted.users`) and rewrites their `roles` array, then
|
|
68
|
+
* deauthenticates them. F-0053 hypothesized this query was globally cross-tenant. It is NOT:
|
|
69
|
+
* `DBUnrestricted` bypasses per-user/per-role RBAC but is STILL scoped to the acting session's
|
|
70
|
+
* `businessId` (see `modifyFilterForAccessConstraint` injecting `{ businessId }` at
|
|
71
|
+
* database.ts:1761-1763, reached via the `unrestricted: true` branch at database.ts:2137-2144).
|
|
72
|
+
*
|
|
73
|
+
* This test locks that boundary in place so a future refactor of `DBUnrestricted` semantics
|
|
74
|
+
* can't silently turn the cascade into a cross-tenant write:
|
|
75
|
+
* 1. Tenant A creates a role `ROLE_OLD` and assigns it to a Tenant A user (positive control).
|
|
76
|
+
* 2. Tenant B (separate businessId) has a user whose roles include the SAME `ROLE_OLD`.
|
|
77
|
+
* 3. Tenant A renames the role `ROLE_OLD` -> `ROLE_NEW`.
|
|
78
|
+
* 4. Assert the Tenant B user's roles are UNCHANGED (still `[ROLE_OLD]`) <-- guards the tenant boundary.
|
|
79
|
+
* 5. Assert the Tenant A user's roles ARE renamed to `[ROLE_NEW]` <-- same-tenant cascade works.
|
|
80
|
+
*
|
|
81
|
+
* Expected on current (correct) code: BOTH assertions pass. A regression that drops the
|
|
82
|
+
* `businessId` scoping would flip assertion #4 to red (Tenant B user becomes `[ROLE_NEW]`).
|
|
83
|
+
*
|
|
84
|
+
* A collision-proof unique role name (timestamped) is used so the test never touches real roles.
|
|
85
|
+
*/
|
|
86
|
+
var cascade_role_rename_cross_tenant_tests = function (_a) {
|
|
87
|
+
var sdk = _a.sdk;
|
|
88
|
+
return __awaiter(void 0, void 0, void 0, function () {
|
|
89
|
+
var stamp, ROLE_OLD, ROLE_NEW, sdkOther, rbapId, controlUserId, victimUserId, tenantABusinessId, rbap, controlUser, victimUser, victimBefore, controlBefore, victimAfter, controlAfter, _b, _c, _d;
|
|
90
|
+
return __generator(this, function (_e) {
|
|
91
|
+
switch (_e.label) {
|
|
92
|
+
case 0:
|
|
93
|
+
(0, testing_1.log_header)("F-0053: cascade role rename cross-tenant regression");
|
|
94
|
+
stamp = Date.now();
|
|
95
|
+
ROLE_OLD = "XTenantRename_".concat(stamp);
|
|
96
|
+
ROLE_NEW = "".concat(ROLE_OLD, "_renamed");
|
|
97
|
+
sdkOther = new sdk_1.Session({ host: host, apiKey: OTHER_TENANT_API_KEY });
|
|
98
|
+
_e.label = 1;
|
|
99
|
+
case 1:
|
|
100
|
+
_e.trys.push([1, , 13, 26]);
|
|
101
|
+
tenantABusinessId = sdk.userInfo.businessId;
|
|
102
|
+
return [4 /*yield*/, sdk.api.role_based_access_permissions.createOne({
|
|
103
|
+
role: ROLE_OLD,
|
|
104
|
+
permissions: __assign({}, constants_1.PROVIDER_PERMISSIONS),
|
|
105
|
+
})];
|
|
106
|
+
case 2:
|
|
107
|
+
rbap = _e.sent();
|
|
108
|
+
rbapId = rbap.id;
|
|
109
|
+
return [4 /*yield*/, sdk.api.users.createOne({
|
|
110
|
+
email: "f0053-control-".concat(stamp, "@example.com"),
|
|
111
|
+
notificationEmailsDisabled: true,
|
|
112
|
+
})];
|
|
113
|
+
case 3:
|
|
114
|
+
controlUser = _e.sent();
|
|
115
|
+
controlUserId = controlUser.id;
|
|
116
|
+
return [4 /*yield*/, sdk.api.users.updateOne(controlUserId, { roles: [ROLE_OLD] }, { replaceObjectFields: true })
|
|
117
|
+
// 3. Tenant B: create a throwaway victim user holding the SAME ROLE_OLD.
|
|
118
|
+
];
|
|
119
|
+
case 4:
|
|
120
|
+
_e.sent();
|
|
121
|
+
return [4 /*yield*/, sdkOther.api.users.createOne({
|
|
122
|
+
email: "f0053-victim-".concat(stamp, "@example.com"),
|
|
123
|
+
notificationEmailsDisabled: true,
|
|
124
|
+
})];
|
|
125
|
+
case 5:
|
|
126
|
+
victimUser = _e.sent();
|
|
127
|
+
victimUserId = victimUser.id;
|
|
128
|
+
return [4 /*yield*/, sdkOther.api.users.updateOne(victimUserId, { roles: [ROLE_OLD] }, { replaceObjectFields: true })
|
|
129
|
+
// 4. Setup sanity: tenants are distinct and both users actually hold ROLE_OLD before the rename.
|
|
130
|
+
// (Distinguishes a setup failure from the security assertion below.)
|
|
131
|
+
];
|
|
132
|
+
case 6:
|
|
133
|
+
_e.sent();
|
|
134
|
+
// 4. Setup sanity: tenants are distinct and both users actually hold ROLE_OLD before the rename.
|
|
135
|
+
// (Distinguishes a setup failure from the security assertion below.)
|
|
136
|
+
(0, testing_1.assert)(victimUser.businessId !== tenantABusinessId, "Victim user shares businessId with Tenant A (".concat(victimUser.businessId, ") \u2014 not a cross-tenant scenario"), 'F-0053 setup: tenants are distinct');
|
|
137
|
+
return [4 /*yield*/, sdkOther.api.users.getOne(victimUserId)];
|
|
138
|
+
case 7:
|
|
139
|
+
victimBefore = _e.sent();
|
|
140
|
+
return [4 /*yield*/, sdk.api.users.getOne(controlUserId)];
|
|
141
|
+
case 8:
|
|
142
|
+
controlBefore = _e.sent();
|
|
143
|
+
(0, testing_1.assert)(JSON.stringify(victimBefore.roles) === JSON.stringify([ROLE_OLD])
|
|
144
|
+
&& JSON.stringify(controlBefore.roles) === JSON.stringify([ROLE_OLD]), "Setup failed: expected both users to hold [".concat(ROLE_OLD, "] (victim=").concat(JSON.stringify(victimBefore.roles), ", control=").concat(JSON.stringify(controlBefore.roles), ")"), 'F-0053 setup: both users hold ROLE_OLD');
|
|
145
|
+
// 5. Tenant A renames the role. This triggers cascade_role_rename.
|
|
146
|
+
return [4 /*yield*/, sdk.api.role_based_access_permissions.updateOne(rbapId, { role: ROLE_NEW })];
|
|
147
|
+
case 9:
|
|
148
|
+
// 5. Tenant A renames the role. This triggers cascade_role_rename.
|
|
149
|
+
_e.sent();
|
|
150
|
+
return [4 /*yield*/, (0, testing_1.wait)(undefined, 1500)
|
|
151
|
+
// 6. SECURITY ASSERTION — the Tenant B victim must be untouched by Tenant A's rename.
|
|
152
|
+
]; // let the side effect run
|
|
153
|
+
case 10:
|
|
154
|
+
_e.sent(); // let the side effect run
|
|
155
|
+
return [4 /*yield*/, sdkOther.api.users.getOne(victimUserId)];
|
|
156
|
+
case 11:
|
|
157
|
+
victimAfter = _e.sent();
|
|
158
|
+
(0, testing_1.assert)(JSON.stringify(victimAfter.roles) === JSON.stringify([ROLE_OLD]), "CROSS-TENANT LEAK: Tenant B victim roles changed to ".concat(JSON.stringify(victimAfter.roles), " ")
|
|
159
|
+
+ "after Tenant A renamed its role. Expected [".concat(ROLE_OLD, "]."), 'F-0053: Tenant B user roles unaffected by other-tenant role rename');
|
|
160
|
+
return [4 /*yield*/, sdk.api.users.getOne(controlUserId)];
|
|
161
|
+
case 12:
|
|
162
|
+
controlAfter = _e.sent();
|
|
163
|
+
(0, testing_1.assert)(JSON.stringify(controlAfter.roles) === JSON.stringify([ROLE_NEW]), "Same-tenant cascade broken: Tenant A control roles are ".concat(JSON.stringify(controlAfter.roles), ", ")
|
|
164
|
+
+ "expected [".concat(ROLE_NEW, "]."), 'F-0053: Tenant A user roles renamed by same-tenant cascade');
|
|
165
|
+
return [3 /*break*/, 26];
|
|
166
|
+
case 13:
|
|
167
|
+
if (!victimUserId) return [3 /*break*/, 17];
|
|
168
|
+
_e.label = 14;
|
|
169
|
+
case 14:
|
|
170
|
+
_e.trys.push([14, 16, , 17]);
|
|
171
|
+
return [4 /*yield*/, sdkOther.api.users.deleteOne(victimUserId)];
|
|
172
|
+
case 15:
|
|
173
|
+
_e.sent();
|
|
174
|
+
return [3 /*break*/, 17];
|
|
175
|
+
case 16:
|
|
176
|
+
_b = _e.sent();
|
|
177
|
+
return [3 /*break*/, 17];
|
|
178
|
+
case 17:
|
|
179
|
+
if (!controlUserId) return [3 /*break*/, 21];
|
|
180
|
+
_e.label = 18;
|
|
181
|
+
case 18:
|
|
182
|
+
_e.trys.push([18, 20, , 21]);
|
|
183
|
+
return [4 /*yield*/, sdk.api.users.deleteOne(controlUserId)];
|
|
184
|
+
case 19:
|
|
185
|
+
_e.sent();
|
|
186
|
+
return [3 /*break*/, 21];
|
|
187
|
+
case 20:
|
|
188
|
+
_c = _e.sent();
|
|
189
|
+
return [3 /*break*/, 21];
|
|
190
|
+
case 21:
|
|
191
|
+
if (!rbapId) return [3 /*break*/, 25];
|
|
192
|
+
_e.label = 22;
|
|
193
|
+
case 22:
|
|
194
|
+
_e.trys.push([22, 24, , 25]);
|
|
195
|
+
return [4 /*yield*/, sdk.api.role_based_access_permissions.deleteOne(rbapId)];
|
|
196
|
+
case 23:
|
|
197
|
+
_e.sent();
|
|
198
|
+
return [3 /*break*/, 25];
|
|
199
|
+
case 24:
|
|
200
|
+
_d = _e.sent();
|
|
201
|
+
return [3 /*break*/, 25];
|
|
202
|
+
case 25: return [7 /*endfinally*/];
|
|
203
|
+
case 26: return [2 /*return*/];
|
|
204
|
+
}
|
|
205
|
+
});
|
|
206
|
+
});
|
|
207
|
+
};
|
|
208
|
+
exports.cascade_role_rename_cross_tenant_tests = cascade_role_rename_cross_tenant_tests;
|
|
209
|
+
// Allow running this test file independently
|
|
210
|
+
if (require.main === module) {
|
|
211
|
+
console.log("\uD83C\uDF10 Using API URL: ".concat(host));
|
|
212
|
+
var sdk_2 = new sdk_1.Session({ host: host });
|
|
213
|
+
var sdkNonAdmin_1 = new sdk_1.Session({ host: host });
|
|
214
|
+
var runTests = function () { return __awaiter(void 0, void 0, void 0, function () {
|
|
215
|
+
return __generator(this, function (_a) {
|
|
216
|
+
switch (_a.label) {
|
|
217
|
+
case 0: return [4 /*yield*/, (0, setup_1.setup_tests)(sdk_2, sdkNonAdmin_1)];
|
|
218
|
+
case 1:
|
|
219
|
+
_a.sent();
|
|
220
|
+
return [4 /*yield*/, (0, exports.cascade_role_rename_cross_tenant_tests)({ sdk: sdk_2, sdkNonAdmin: sdkNonAdmin_1 })];
|
|
221
|
+
case 2:
|
|
222
|
+
_a.sent();
|
|
223
|
+
return [2 /*return*/];
|
|
224
|
+
}
|
|
225
|
+
});
|
|
226
|
+
}); };
|
|
227
|
+
runTests()
|
|
228
|
+
.then(function () {
|
|
229
|
+
console.log("✅ F-0053 cascade role rename cross-tenant test suite completed successfully");
|
|
230
|
+
process.exit(0);
|
|
231
|
+
})
|
|
232
|
+
.catch(function (error) {
|
|
233
|
+
console.error("❌ F-0053 cascade role rename cross-tenant test suite failed:", error);
|
|
234
|
+
process.exit(1);
|
|
235
|
+
});
|
|
236
|
+
}
|
|
237
|
+
//# sourceMappingURL=F-0053-cascade-role-rename-cross-tenant.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"F-0053-cascade-role-rename-cross-tenant.test.js","sourceRoot":"","sources":["../../../../../src/tests/api_tests/security/F-0053-cascade-role-rename-cross-tenant.test.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA,OAAO,CAAC,oBAAoB,CAAC,CAAC,OAAO,EAAE,CAAC;AAExC,oCAAsC;AACtC,+CAI4B;AAC5B,qCAAyC;AACzC,mDAA4D;AAE5D,IAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,OAAO,IAAI,uBAAgC,CAAA;AAEpE,iFAAiF;AACjF,4CAA4C;AAC5C,IAAM,oBAAoB,GAAG,kEAAkE,CAAA;AAE/F;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACI,IAAM,sCAAsC,GAAG,UAAO,EAAgD;QAA9C,GAAG,SAAA;;;;;;oBAChE,IAAA,oBAAU,EAAC,qDAAqD,CAAC,CAAA;oBAE3D,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAA;oBAClB,QAAQ,GAAG,wBAAiB,KAAK,CAAE,CAAA;oBACnC,QAAQ,GAAG,UAAG,QAAQ,aAAU,CAAA;oBAIhC,QAAQ,GAAG,IAAI,aAAO,CAAC,EAAE,IAAI,MAAA,EAAE,MAAM,EAAE,oBAAoB,EAAE,CAAC,CAAA;;;;oBAY5D,iBAAiB,GAAG,GAAG,CAAC,QAAQ,CAAC,UAAU,CAAA;oBAGpC,qBAAM,GAAG,CAAC,GAAG,CAAC,6BAA6B,CAAC,SAAS,CAAC;4BACjE,IAAI,EAAE,QAAQ;4BACd,WAAW,eAAO,gCAAoB,CAAE;yBACzC,CAAC,EAAA;;oBAHI,IAAI,GAAG,SAGX;oBACF,MAAM,GAAG,IAAI,CAAC,EAAE,CAAA;oBAGI,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC;4BAChD,KAAK,EAAE,wBAAiB,KAAK,iBAAc;4BAC3C,0BAA0B,EAAE,IAAI;yBAC1B,CAAC,EAAA;;oBAHH,WAAW,GAAG,SAGX;oBACT,aAAa,GAAG,WAAW,CAAC,EAAE,CAAA;oBAC9B,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,aAAa,EAAE,EAAE,KAAK,EAAE,CAAC,QAAQ,CAAC,EAAE,EAAE,EAAE,mBAAmB,EAAE,IAAI,EAAE,CAAC;wBAElG,yEAAyE;sBAFyB;;oBAAlG,SAAkG,CAAA;oBAG/E,qBAAM,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC;4BACpD,KAAK,EAAE,uBAAgB,KAAK,iBAAc;4BAC1C,0BAA0B,EAAE,IAAI;yBAC1B,CAAC,EAAA;;oBAHH,UAAU,GAAG,SAGV;oBACT,YAAY,GAAG,UAAU,CAAC,EAAE,CAAA;oBAC5B,qBAAM,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,YAAY,EAAE,EAAE,KAAK,EAAE,CAAC,QAAQ,CAAC,EAAE,EAAE,EAAE,mBAAmB,EAAE,IAAI,EAAE,CAAC;wBAEtG,iGAAiG;wBACjG,wEAAwE;sBAH8B;;oBAAtG,SAAsG,CAAA;oBAEtG,iGAAiG;oBACjG,wEAAwE;oBACxE,IAAA,gBAAM,EACJ,UAAU,CAAC,UAAU,KAAK,iBAAiB,EAC3C,uDAAgD,UAAU,CAAC,UAAU,yCAAiC,EACtG,oCAAoC,CACrC,CAAA;oBACoB,qBAAM,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,YAAY,CAAC,EAAA;;oBAA5D,YAAY,GAAG,SAA6C;oBAC5C,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,aAAa,CAAC,EAAA;;oBAAzD,aAAa,GAAG,SAAyC;oBAC/D,IAAA,gBAAM,EACJ,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,CAAC,QAAQ,CAAC,CAAC;2BAC5D,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,CAAC,QAAQ,CAAC,CAAC,EACvE,qDAA8C,QAAQ,uBAAa,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,KAAK,CAAC,uBAAa,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC,KAAK,CAAC,MAAG,EACxJ,wCAAwC,CACzC,CAAA;oBAED,mEAAmE;oBACnE,qBAAM,GAAG,CAAC,GAAG,CAAC,6BAA6B,CAAC,SAAS,CAAC,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,EAAA;;oBADjF,mEAAmE;oBACnE,SAAiF,CAAA;oBACjF,qBAAM,IAAA,cAAI,EAAC,SAAS,EAAE,IAAI,CAAC;wBAE3B,sFAAsF;sBAF3D,CAAC,0BAA0B;;oBAAtD,SAA2B,CAAA,CAAC,0BAA0B;oBAGlC,qBAAM,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,YAAY,CAAC,EAAA;;oBAA3D,WAAW,GAAG,SAA6C;oBACjE,IAAA,gBAAM,EACJ,IAAI,CAAC,SAAS,CAAC,WAAW,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,CAAC,QAAQ,CAAC,CAAC,EAChE,8DAAuD,IAAI,CAAC,SAAS,CAAC,WAAW,CAAC,KAAK,CAAC,MAAG;0BACvF,qDAA8C,QAAQ,OAAI,EAC9D,oEAAoE,CACrE,CAAA;oBAGoB,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,aAAa,CAAC,EAAA;;oBAAxD,YAAY,GAAG,SAAyC;oBAC9D,IAAA,gBAAM,EACJ,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,CAAC,QAAQ,CAAC,CAAC,EACjE,iEAA0D,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,KAAK,CAAC,OAAI;0BAC5F,oBAAa,QAAQ,OAAI,EAC7B,4DAA4D,CAC7D,CAAA;;;yBAGG,YAAY,EAAZ,yBAAY;;;;oBACR,qBAAM,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,YAAY,CAAC,EAAA;;oBAAhD,SAAgD,CAAA;;;;;;yBAEpD,aAAa,EAAb,yBAAa;;;;oBACT,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,aAAa,CAAC,EAAA;;oBAA5C,SAA4C,CAAA;;;;;;yBAEhD,MAAM,EAAN,yBAAM;;;;oBACF,qBAAM,GAAG,CAAC,GAAG,CAAC,6BAA6B,CAAC,SAAS,CAAC,MAAM,CAAC,EAAA;;oBAA7D,SAA6D,CAAA;;;;;;;;;;CAGxE,CAAA;AA/FY,QAAA,sCAAsC,0CA+FlD;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,8CAAsC,EAAC,EAAE,GAAG,OAAA,EAAE,WAAW,eAAA,EAAE,CAAC,EAAA;;oBAAlE,SAAkE,CAAA;;;;SACnE,CAAA;IAED,QAAQ,EAAE;SACP,IAAI,CAAC;QACJ,OAAO,CAAC,GAAG,CAAC,6EAA6E,CAAC,CAAA;QAC1F,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC,CAAC;SACD,KAAK,CAAC,UAAC,KAAK;QACX,OAAO,CAAC,KAAK,CAAC,8DAA8D,EAAE,KAAK,CAAC,CAAA;QACpF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC,CAAC,CAAA;CACL"}
|
|
@@ -0,0 +1,38 @@
|
|
|
1
|
+
import { Session } from "../../../sdk";
|
|
2
|
+
/**
|
|
3
|
+
* Self-role privilege-escalation guard (relates to security-audit finding F-0076, which was
|
|
4
|
+
* investigated and closed as a FALSE POSITIVE — see that file for the full code trace).
|
|
5
|
+
*
|
|
6
|
+
* F-0076 hypothesized that a non-admin staff user could `PATCH /v1/users/{their-own-id}` with
|
|
7
|
+
* `{ roles: ['Admin'] }` and self-promote to Admin, because the FIRST `users` relationship
|
|
8
|
+
* constraint ("Only admin users can set the admin role",
|
|
9
|
+
* [schema.ts:3446](packages/public/schema/src/schema.ts#L3446)) has a self-exception
|
|
10
|
+
* (`if (_id === session.id) return`).
|
|
11
|
+
*
|
|
12
|
+
* That analysis missed the SECOND constraint, "Only admin users can update user roles"
|
|
13
|
+
* ([schema.ts:3486](packages/public/schema/src/schema.ts#L3486)), which has NO self-exception.
|
|
14
|
+
* Relationship constraints are AND-evaluated — `validateRelationshipConstraints`
|
|
15
|
+
* ([routing.ts:1240-1252](packages/private/api/api/modules/routing.ts#L1240)) loops the whole
|
|
16
|
+
* array and throws 400 on the FIRST evaluator that returns a string. So a non-admin self-update
|
|
17
|
+
* that includes `roles` passes constraint #1 (self-exception) but is rejected by constraint #2.
|
|
18
|
+
* The self-promotion is blocked.
|
|
19
|
+
*
|
|
20
|
+
* This test locks that boundary in place so a future refactor of the role constraints can't
|
|
21
|
+
* silently reintroduce the escalation. A dedicated throwaway non-admin user is used as the
|
|
22
|
+
* "attacker" (we never mutate the shared sdkNonAdmin's roles):
|
|
23
|
+
* 1. Admin creates a throwaway user and assigns it a non-admin role (`['Provider']`).
|
|
24
|
+
* 2. Authenticate AS that user via a freshly-minted auth token.
|
|
25
|
+
* 3. As the attacker, attempt four self-role mutations — ['Admin'], ['Provider','Admin'],
|
|
26
|
+
* an arbitrary role, and [] — and assert EACH is blocked. <-- the security assertions
|
|
27
|
+
* 4. Confirm (as admin) the attacker's roles are still ['Provider'] — nothing slipped through.
|
|
28
|
+
* 5. Positive control: admin CAN update the throwaway user's roles. <-- guards against an
|
|
29
|
+
* over-restrictive regression that would block legitimate admin role management.
|
|
30
|
+
*
|
|
31
|
+
* Expected on current (correct) code: all assertions pass. A regression that made the self-update
|
|
32
|
+
* path role-writable by non-admins would flip the step-3/step-4 assertions to red.
|
|
33
|
+
*/
|
|
34
|
+
export declare const self_admin_role_assignment_tests: ({ sdk }: {
|
|
35
|
+
sdk: Session;
|
|
36
|
+
sdkNonAdmin: Session;
|
|
37
|
+
}) => Promise<void>;
|
|
38
|
+
//# sourceMappingURL=F-0076-self-admin-role-assignment.test.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"F-0076-self-admin-role-assignment.test.d.ts","sourceRoot":"","sources":["../../../../../src/tests/api_tests/security/F-0076-self-admin-role-assignment.test.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,OAAO,EAAE,MAAM,cAAc,CAAA;AAUtC;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,eAAO,MAAM,gCAAgC;SAA2B,OAAO;iBAAe,OAAO;mBAkGpG,CAAA"}
|
|
@@ -0,0 +1,222 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
|
|
3
|
+
function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
|
|
4
|
+
return new (P || (P = Promise))(function (resolve, reject) {
|
|
5
|
+
function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
|
|
6
|
+
function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
|
|
7
|
+
function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
|
|
8
|
+
step((generator = generator.apply(thisArg, _arguments || [])).next());
|
|
9
|
+
});
|
|
10
|
+
};
|
|
11
|
+
var __generator = (this && this.__generator) || function (thisArg, body) {
|
|
12
|
+
var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g;
|
|
13
|
+
return g = { next: verb(0), "throw": verb(1), "return": verb(2) }, typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
|
|
14
|
+
function verb(n) { return function (v) { return step([n, v]); }; }
|
|
15
|
+
function step(op) {
|
|
16
|
+
if (f) throw new TypeError("Generator is already executing.");
|
|
17
|
+
while (g && (g = 0, op[0] && (_ = 0)), _) try {
|
|
18
|
+
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;
|
|
19
|
+
if (y = 0, t) op = [op[0] & 2, t.value];
|
|
20
|
+
switch (op[0]) {
|
|
21
|
+
case 0: case 1: t = op; break;
|
|
22
|
+
case 4: _.label++; return { value: op[1], done: false };
|
|
23
|
+
case 5: _.label++; y = op[1]; op = [0]; continue;
|
|
24
|
+
case 7: op = _.ops.pop(); _.trys.pop(); continue;
|
|
25
|
+
default:
|
|
26
|
+
if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
|
|
27
|
+
if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
|
|
28
|
+
if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
|
|
29
|
+
if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
|
|
30
|
+
if (t[2]) _.ops.pop();
|
|
31
|
+
_.trys.pop(); continue;
|
|
32
|
+
}
|
|
33
|
+
op = body.call(thisArg, _);
|
|
34
|
+
} catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
|
|
35
|
+
if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
|
|
36
|
+
}
|
|
37
|
+
};
|
|
38
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
39
|
+
exports.self_admin_role_assignment_tests = void 0;
|
|
40
|
+
require('source-map-support').install();
|
|
41
|
+
var sdk_1 = require("../../../sdk");
|
|
42
|
+
var testing_1 = require("@tellescope/testing");
|
|
43
|
+
var setup_1 = require("../../setup");
|
|
44
|
+
var host = process.env.API_URL || 'http://localhost:8080';
|
|
45
|
+
/**
|
|
46
|
+
* Self-role privilege-escalation guard (relates to security-audit finding F-0076, which was
|
|
47
|
+
* investigated and closed as a FALSE POSITIVE — see that file for the full code trace).
|
|
48
|
+
*
|
|
49
|
+
* F-0076 hypothesized that a non-admin staff user could `PATCH /v1/users/{their-own-id}` with
|
|
50
|
+
* `{ roles: ['Admin'] }` and self-promote to Admin, because the FIRST `users` relationship
|
|
51
|
+
* constraint ("Only admin users can set the admin role",
|
|
52
|
+
* [schema.ts:3446](packages/public/schema/src/schema.ts#L3446)) has a self-exception
|
|
53
|
+
* (`if (_id === session.id) return`).
|
|
54
|
+
*
|
|
55
|
+
* That analysis missed the SECOND constraint, "Only admin users can update user roles"
|
|
56
|
+
* ([schema.ts:3486](packages/public/schema/src/schema.ts#L3486)), which has NO self-exception.
|
|
57
|
+
* Relationship constraints are AND-evaluated — `validateRelationshipConstraints`
|
|
58
|
+
* ([routing.ts:1240-1252](packages/private/api/api/modules/routing.ts#L1240)) loops the whole
|
|
59
|
+
* array and throws 400 on the FIRST evaluator that returns a string. So a non-admin self-update
|
|
60
|
+
* that includes `roles` passes constraint #1 (self-exception) but is rejected by constraint #2.
|
|
61
|
+
* The self-promotion is blocked.
|
|
62
|
+
*
|
|
63
|
+
* This test locks that boundary in place so a future refactor of the role constraints can't
|
|
64
|
+
* silently reintroduce the escalation. A dedicated throwaway non-admin user is used as the
|
|
65
|
+
* "attacker" (we never mutate the shared sdkNonAdmin's roles):
|
|
66
|
+
* 1. Admin creates a throwaway user and assigns it a non-admin role (`['Provider']`).
|
|
67
|
+
* 2. Authenticate AS that user via a freshly-minted auth token.
|
|
68
|
+
* 3. As the attacker, attempt four self-role mutations — ['Admin'], ['Provider','Admin'],
|
|
69
|
+
* an arbitrary role, and [] — and assert EACH is blocked. <-- the security assertions
|
|
70
|
+
* 4. Confirm (as admin) the attacker's roles are still ['Provider'] — nothing slipped through.
|
|
71
|
+
* 5. Positive control: admin CAN update the throwaway user's roles. <-- guards against an
|
|
72
|
+
* over-restrictive regression that would block legitimate admin role management.
|
|
73
|
+
*
|
|
74
|
+
* Expected on current (correct) code: all assertions pass. A regression that made the self-update
|
|
75
|
+
* path role-writable by non-admins would flip the step-3/step-4 assertions to red.
|
|
76
|
+
*/
|
|
77
|
+
var self_admin_role_assignment_tests = function (_a) {
|
|
78
|
+
var sdk = _a.sdk;
|
|
79
|
+
return __awaiter(void 0, void 0, void 0, function () {
|
|
80
|
+
var stamp, NON_ADMIN_ROLE, expect_blocked, attackerId, attacker, attackerBefore, sdkAttacker_1, _b, attackerAfter, afterAdminUpdate, _c;
|
|
81
|
+
var _d;
|
|
82
|
+
var _e;
|
|
83
|
+
return __generator(this, function (_f) {
|
|
84
|
+
switch (_f.label) {
|
|
85
|
+
case 0:
|
|
86
|
+
(0, testing_1.log_header)("F-0076: self-admin role assignment privilege-escalation regression");
|
|
87
|
+
stamp = Date.now();
|
|
88
|
+
NON_ADMIN_ROLE = 'Provider';
|
|
89
|
+
expect_blocked = function (fn, description) { return __awaiter(void 0, void 0, void 0, function () {
|
|
90
|
+
var e_1;
|
|
91
|
+
return __generator(this, function (_a) {
|
|
92
|
+
switch (_a.label) {
|
|
93
|
+
case 0:
|
|
94
|
+
_a.trys.push([0, 2, , 3]);
|
|
95
|
+
return [4 /*yield*/, fn()];
|
|
96
|
+
case 1:
|
|
97
|
+
_a.sent();
|
|
98
|
+
(0, testing_1.assert)(false, "".concat(description, " - SELF-ROLE ESCALATION SUCCEEDED (expected it to be blocked)"));
|
|
99
|
+
return [3 /*break*/, 3];
|
|
100
|
+
case 2:
|
|
101
|
+
e_1 = _a.sent();
|
|
102
|
+
// CRUD relationship-constraint failures surface as 400 { message, info } via SDK parseError.
|
|
103
|
+
(0, testing_1.assert)((e_1 === null || e_1 === void 0 ? void 0 : e_1.code) === 400 || (e_1 === null || e_1 === void 0 ? void 0 : e_1.statusCode) === 400 || typeof (e_1 === null || e_1 === void 0 ? void 0 : e_1.message) === 'string', "".concat(description, " - expected a block error, got: ").concat(JSON.stringify(e_1)), description);
|
|
104
|
+
return [3 /*break*/, 3];
|
|
105
|
+
case 3: return [2 /*return*/];
|
|
106
|
+
}
|
|
107
|
+
});
|
|
108
|
+
}); };
|
|
109
|
+
_f.label = 1;
|
|
110
|
+
case 1:
|
|
111
|
+
_f.trys.push([1, , 15, 20]);
|
|
112
|
+
return [4 /*yield*/, sdk.api.users.createOne({
|
|
113
|
+
email: "f0076-attacker-".concat(stamp, "@example.com"),
|
|
114
|
+
notificationEmailsDisabled: true,
|
|
115
|
+
verifiedEmail: true,
|
|
116
|
+
})];
|
|
117
|
+
case 2:
|
|
118
|
+
attacker = _f.sent();
|
|
119
|
+
attackerId = attacker.id;
|
|
120
|
+
return [4 /*yield*/, sdk.api.users.updateOne(attackerId, { roles: [NON_ADMIN_ROLE] }, { replaceObjectFields: true })];
|
|
121
|
+
case 3:
|
|
122
|
+
_f.sent();
|
|
123
|
+
return [4 /*yield*/, (0, testing_1.wait)(undefined, 2000)
|
|
124
|
+
// Setup sanity: the attacker holds exactly the non-admin role and is NOT an admin.
|
|
125
|
+
]; // role change triggers a logout; let it propagate before minting a token
|
|
126
|
+
case 4:
|
|
127
|
+
_f.sent(); // role change triggers a logout; let it propagate before minting a token
|
|
128
|
+
return [4 /*yield*/, sdk.api.users.getOne(attackerId)];
|
|
129
|
+
case 5:
|
|
130
|
+
attackerBefore = _f.sent();
|
|
131
|
+
(0, testing_1.assert)(JSON.stringify(attackerBefore.roles) === JSON.stringify([NON_ADMIN_ROLE]), "Setup failed: expected attacker to hold [".concat(NON_ADMIN_ROLE, "], got ").concat(JSON.stringify(attackerBefore.roles)), 'F-0076 setup: attacker holds a non-admin role');
|
|
132
|
+
_b = sdk_1.Session.bind;
|
|
133
|
+
_d = {
|
|
134
|
+
host: host
|
|
135
|
+
};
|
|
136
|
+
return [4 /*yield*/, sdk.api.users.generate_auth_token({ id: attackerId })];
|
|
137
|
+
case 6:
|
|
138
|
+
sdkAttacker_1 = new (_b.apply(sdk_1.Session, [void 0, (_d.authToken = (_f.sent()).authToken,
|
|
139
|
+
_d)]))();
|
|
140
|
+
return [4 /*yield*/, sdkAttacker_1.refresh_session()]; // populate userInfo from the freshly-minted token
|
|
141
|
+
case 7:
|
|
142
|
+
_f.sent(); // populate userInfo from the freshly-minted token
|
|
143
|
+
(0, testing_1.assert)(sdkAttacker_1.userInfo.id === attackerId && !((_e = sdkAttacker_1.userInfo.roles) !== null && _e !== void 0 ? _e : []).includes('Admin'), "Setup failed: attacker session is not the expected non-admin user", 'F-0076 setup: authenticated as the non-admin attacker');
|
|
144
|
+
// 3. SECURITY ASSERTIONS — every self-role mutation by the non-admin must be blocked.
|
|
145
|
+
return [4 /*yield*/, expect_blocked(function () { return sdkAttacker_1.api.users.updateOne(attackerId, { roles: ['Admin'] }, { replaceObjectFields: true }); }, 'F-0076: non-admin self-update to [Admin] is blocked')];
|
|
146
|
+
case 8:
|
|
147
|
+
// 3. SECURITY ASSERTIONS — every self-role mutation by the non-admin must be blocked.
|
|
148
|
+
_f.sent();
|
|
149
|
+
return [4 /*yield*/, expect_blocked(function () { return sdkAttacker_1.api.users.updateOne(attackerId, { roles: [NON_ADMIN_ROLE, 'Admin'] }, { replaceObjectFields: true }); }, 'F-0076: non-admin self-update to [Provider, Admin] is blocked')];
|
|
150
|
+
case 9:
|
|
151
|
+
_f.sent();
|
|
152
|
+
return [4 /*yield*/, expect_blocked(function () { return sdkAttacker_1.api.users.updateOne(attackerId, { roles: ["Arbitrary_".concat(stamp)] }, { replaceObjectFields: true }); }, 'F-0076: non-admin self-update to an arbitrary role is blocked')];
|
|
153
|
+
case 10:
|
|
154
|
+
_f.sent();
|
|
155
|
+
return [4 /*yield*/, expect_blocked(function () { return sdkAttacker_1.api.users.updateOne(attackerId, { roles: [] }, { replaceObjectFields: true }); }, 'F-0076: non-admin self-update to [] (would grant defaults) is blocked')
|
|
156
|
+
// 4. STATE ASSERTION — nothing slipped through; roles are still the original non-admin role.
|
|
157
|
+
];
|
|
158
|
+
case 11:
|
|
159
|
+
_f.sent();
|
|
160
|
+
return [4 /*yield*/, sdk.api.users.getOne(attackerId)];
|
|
161
|
+
case 12:
|
|
162
|
+
attackerAfter = _f.sent();
|
|
163
|
+
(0, testing_1.assert)(JSON.stringify(attackerAfter.roles) === JSON.stringify([NON_ADMIN_ROLE]), "ESCALATION LEAK: attacker roles changed to ".concat(JSON.stringify(attackerAfter.roles), " ")
|
|
164
|
+
+ "after self-update attempts. Expected [".concat(NON_ADMIN_ROLE, "]."), 'F-0076: attacker roles unchanged after all self-escalation attempts');
|
|
165
|
+
// 5. POSITIVE CONTROL — an Admin CAN update the user's roles (mechanism is not over-restricted).
|
|
166
|
+
return [4 /*yield*/, sdk.api.users.updateOne(attackerId, { roles: [NON_ADMIN_ROLE] }, { replaceObjectFields: true })];
|
|
167
|
+
case 13:
|
|
168
|
+
// 5. POSITIVE CONTROL — an Admin CAN update the user's roles (mechanism is not over-restricted).
|
|
169
|
+
_f.sent();
|
|
170
|
+
return [4 /*yield*/, sdk.api.users.getOne(attackerId)];
|
|
171
|
+
case 14:
|
|
172
|
+
afterAdminUpdate = _f.sent();
|
|
173
|
+
(0, testing_1.assert)(JSON.stringify(afterAdminUpdate.roles) === JSON.stringify([NON_ADMIN_ROLE]), "Admin role update failed: roles are ".concat(JSON.stringify(afterAdminUpdate.roles), ", expected [").concat(NON_ADMIN_ROLE, "]"), 'F-0076: admin can manage user roles (positive control)');
|
|
174
|
+
return [3 /*break*/, 20];
|
|
175
|
+
case 15:
|
|
176
|
+
if (!attackerId) return [3 /*break*/, 19];
|
|
177
|
+
_f.label = 16;
|
|
178
|
+
case 16:
|
|
179
|
+
_f.trys.push([16, 18, , 19]);
|
|
180
|
+
return [4 /*yield*/, sdk.api.users.deleteOne(attackerId)];
|
|
181
|
+
case 17:
|
|
182
|
+
_f.sent();
|
|
183
|
+
return [3 /*break*/, 19];
|
|
184
|
+
case 18:
|
|
185
|
+
_c = _f.sent();
|
|
186
|
+
return [3 /*break*/, 19];
|
|
187
|
+
case 19: return [7 /*endfinally*/];
|
|
188
|
+
case 20: return [2 /*return*/];
|
|
189
|
+
}
|
|
190
|
+
});
|
|
191
|
+
});
|
|
192
|
+
};
|
|
193
|
+
exports.self_admin_role_assignment_tests = self_admin_role_assignment_tests;
|
|
194
|
+
// Allow running this test file independently
|
|
195
|
+
if (require.main === module) {
|
|
196
|
+
console.log("\uD83C\uDF10 Using API URL: ".concat(host));
|
|
197
|
+
var sdk_2 = new sdk_1.Session({ host: host });
|
|
198
|
+
var sdkNonAdmin_1 = new sdk_1.Session({ host: host });
|
|
199
|
+
var runTests = function () { return __awaiter(void 0, void 0, void 0, function () {
|
|
200
|
+
return __generator(this, function (_a) {
|
|
201
|
+
switch (_a.label) {
|
|
202
|
+
case 0: return [4 /*yield*/, (0, setup_1.setup_tests)(sdk_2, sdkNonAdmin_1)];
|
|
203
|
+
case 1:
|
|
204
|
+
_a.sent();
|
|
205
|
+
return [4 /*yield*/, (0, exports.self_admin_role_assignment_tests)({ sdk: sdk_2, sdkNonAdmin: sdkNonAdmin_1 })];
|
|
206
|
+
case 2:
|
|
207
|
+
_a.sent();
|
|
208
|
+
return [2 /*return*/];
|
|
209
|
+
}
|
|
210
|
+
});
|
|
211
|
+
}); };
|
|
212
|
+
runTests()
|
|
213
|
+
.then(function () {
|
|
214
|
+
console.log("✅ F-0076 self-admin role assignment test suite completed successfully");
|
|
215
|
+
process.exit(0);
|
|
216
|
+
})
|
|
217
|
+
.catch(function (error) {
|
|
218
|
+
console.error("❌ F-0076 self-admin role assignment test suite failed:", error);
|
|
219
|
+
process.exit(1);
|
|
220
|
+
});
|
|
221
|
+
}
|
|
222
|
+
//# sourceMappingURL=F-0076-self-admin-role-assignment.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"F-0076-self-admin-role-assignment.test.js","sourceRoot":"","sources":["../../../../../src/tests/api_tests/security/F-0076-self-admin-role-assignment.test.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA,OAAO,CAAC,oBAAoB,CAAC,CAAC,OAAO,EAAE,CAAC;AAExC,oCAAsC;AACtC,+CAI4B;AAC5B,qCAAyC;AAEzC,IAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,OAAO,IAAI,uBAAgC,CAAA;AAEpE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACI,IAAM,gCAAgC,GAAG,UAAO,EAAgD;QAA9C,GAAG,SAAA;;;;;;;;oBAC1D,IAAA,oBAAU,EAAC,oEAAoE,CAAC,CAAA;oBAE1E,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAA;oBAClB,cAAc,GAAG,UAAU,CAAA;oBAG3B,cAAc,GAAG,UAAO,EAAsB,EAAE,WAAmB;;;;;;oCAErE,qBAAM,EAAE,EAAE,EAAA;;oCAAV,SAAU,CAAA;oCACV,IAAA,gBAAM,EAAC,KAAK,EAAE,UAAG,WAAW,kEAA+D,CAAC,CAAA;;;;oCAE5F,6FAA6F;oCAC7F,IAAA,gBAAM,EACJ,CAAA,GAAC,aAAD,GAAC,uBAAD,GAAC,CAAE,IAAI,MAAK,GAAG,IAAI,CAAA,GAAC,aAAD,GAAC,uBAAD,GAAC,CAAE,UAAU,MAAK,GAAG,IAAI,OAAO,CAAA,GAAC,aAAD,GAAC,uBAAD,GAAC,CAAE,OAAO,CAAA,KAAK,QAAQ,EAC1E,UAAG,WAAW,6CAAmC,IAAI,CAAC,SAAS,CAAC,GAAC,CAAC,CAAE,EACpE,WAAW,CACZ,CAAA;;;;;yBAEJ,CAAA;;;;oBASkB,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC;4BAC7C,KAAK,EAAE,yBAAkB,KAAK,iBAAc;4BAC5C,0BAA0B,EAAE,IAAI;4BAChC,aAAa,EAAE,IAAI;yBACb,CAAC,EAAA;;oBAJH,QAAQ,GAAG,SAIR;oBACT,UAAU,GAAG,QAAQ,CAAC,EAAE,CAAA;oBACxB,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,UAAU,EAAE,EAAE,KAAK,EAAE,CAAC,cAAc,CAAC,EAAE,EAAE,EAAE,mBAAmB,EAAE,IAAI,EAAE,CAAC,EAAA;;oBAArG,SAAqG,CAAA;oBACrG,qBAAM,IAAA,cAAI,EAAC,SAAS,EAAE,IAAI,CAAC;wBAE3B,mFAAmF;sBAFxD,CAAC,yEAAyE;;oBAArG,SAA2B,CAAA,CAAC,yEAAyE;oBAG9E,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,UAAU,CAAC,EAAA;;oBAAvD,cAAc,GAAG,SAAsC;oBAC7D,IAAA,gBAAM,EACJ,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,CAAC,cAAc,CAAC,CAAC,EACzE,mDAA4C,cAAc,oBAAU,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,KAAK,CAAC,CAAE,EAC1G,+CAA+C,CAChD,CAAA;yBAGuB,aAAO;;wBAC7B,IAAI,MAAA;;oBACQ,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,mBAAmB,CAAC,EAAE,EAAE,EAAE,UAAU,EAAE,CAAC,EAAA;;oBAFnE,gBAAc,cAAI,aAAO,YAE7B,YAAS,GAAE,CAAC,SAA2D,CAAC,CAAC,SAAS;oCAClF;oBACF,qBAAM,aAAW,CAAC,eAAe,EAAE,EAAA,CAAC,kDAAkD;;oBAAtF,SAAmC,CAAA,CAAC,kDAAkD;oBACtF,IAAA,gBAAM,EACJ,aAAW,CAAC,QAAQ,CAAC,EAAE,KAAK,UAAU,IAAI,CAAC,CAAC,MAAA,aAAW,CAAC,QAAQ,CAAC,KAAK,mCAAI,EAAE,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,EAC/F,mEAAmE,EACnE,uDAAuD,CACxD,CAAA;oBAED,sFAAsF;oBACtF,qBAAM,cAAc,CAClB,cAAM,OAAA,aAAW,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,UAAW,EAAE,EAAE,KAAK,EAAE,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,mBAAmB,EAAE,IAAI,EAAE,CAAC,EAAjG,CAAiG,EACvG,qDAAqD,CACtD,EAAA;;oBAJD,sFAAsF;oBACtF,SAGC,CAAA;oBACD,qBAAM,cAAc,CAClB,cAAM,OAAA,aAAW,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,UAAW,EAAE,EAAE,KAAK,EAAE,CAAC,cAAc,EAAE,OAAO,CAAC,EAAE,EAAE,EAAE,mBAAmB,EAAE,IAAI,EAAE,CAAC,EAAjH,CAAiH,EACvH,+DAA+D,CAChE,EAAA;;oBAHD,SAGC,CAAA;oBACD,qBAAM,cAAc,CAClB,cAAM,OAAA,aAAW,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,UAAW,EAAE,EAAE,KAAK,EAAE,CAAC,oBAAa,KAAK,CAAE,CAAC,EAAE,EAAE,EAAE,mBAAmB,EAAE,IAAI,EAAE,CAAC,EAA9G,CAA8G,EACpH,+DAA+D,CAChE,EAAA;;oBAHD,SAGC,CAAA;oBACD,qBAAM,cAAc,CAClB,cAAM,OAAA,aAAW,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,UAAW,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,EAAE,EAAE,mBAAmB,EAAE,IAAI,EAAE,CAAC,EAA1F,CAA0F,EAChG,uEAAuE,CACxE;wBAED,6FAA6F;sBAF5F;;oBAHD,SAGC,CAAA;oBAGqB,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,UAAU,CAAC,EAAA;;oBAAtD,aAAa,GAAG,SAAsC;oBAC5D,IAAA,gBAAM,EACJ,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,CAAC,cAAc,CAAC,CAAC,EACxE,qDAA8C,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC,KAAK,CAAC,MAAG;0BAChF,gDAAyC,cAAc,OAAI,EAC/D,qEAAqE,CACtE,CAAA;oBAED,iGAAiG;oBACjG,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,UAAU,EAAE,EAAE,KAAK,EAAE,CAAC,cAAc,CAAC,EAAE,EAAE,EAAE,mBAAmB,EAAE,IAAI,EAAE,CAAC,EAAA;;oBADrG,iGAAiG;oBACjG,SAAqG,CAAA;oBAC5E,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,UAAU,CAAC,EAAA;;oBAAzD,gBAAgB,GAAG,SAAsC;oBAC/D,IAAA,gBAAM,EACJ,IAAI,CAAC,SAAS,CAAC,gBAAgB,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,CAAC,cAAc,CAAC,CAAC,EAC3E,8CAAuC,IAAI,CAAC,SAAS,CAAC,gBAAgB,CAAC,KAAK,CAAC,yBAAe,cAAc,MAAG,EAC7G,wDAAwD,CACzD,CAAA;;;yBAGG,UAAU,EAAV,yBAAU;;;;oBACN,qBAAM,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,UAAU,CAAC,EAAA;;oBAAzC,SAAyC,CAAA;;;;;;;;;;CAGpD,CAAA;AAlGY,QAAA,gCAAgC,oCAkG5C;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,wCAAgC,EAAC,EAAE,GAAG,OAAA,EAAE,WAAW,eAAA,EAAE,CAAC,EAAA;;oBAA5D,SAA4D,CAAA;;;;SAC7D,CAAA;IAED,QAAQ,EAAE;SACP,IAAI,CAAC;QACJ,OAAO,CAAC,GAAG,CAAC,uEAAuE,CAAC,CAAA;QACpF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC,CAAC;SACD,KAAK,CAAC,UAAC,KAAK;QACX,OAAO,CAAC,KAAK,CAAC,wDAAwD,EAAE,KAAK,CAAC,CAAA;QAC9E,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC,CAAC,CAAA;CACL"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"user_portal_settings.test.d.ts","sourceRoot":"","sources":["../../../../src/tests/api_tests/user_portal_settings.test.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,OAAO,EAAkB,MAAM,WAAW,CAAA;AAanD,eAAO,MAAM,0BAA0B;SAAuC,OAAO;iBAAe,OAAO;mBAmL1G,CAAA"}
|