@7365admin1/core 3.52.16-staging.266 → 3.52.16-staging.267

package/.changeset/user-email-allowlist-subscription-authz.md

@@ -0,0 +1,21 @@
+---
+"@7365admin1/core": patch
+---
+
+Narrow the account-by-e-mail lookup to identity only, and scope the two open subscription reads
+
+`GET /api/users/email/:email` and its v2 twin `GET /api/users/v2/email/:email`
+stripped the password hash and session id from the reply but answered every
+other stored field, so any signed-in account could read any other account's
+NRIC, contact number, date of birth, gender, default organisation, status and
+profile just by knowing the e-mail address — across every client on the
+platform. Both now answer `{_id, name, email}` and nothing else. An
+organisation gate would have been the wrong fix: this endpoint exists for the
+invite flow, where the person being looked up is deliberately not in the
+caller's organisation yet.
+
+`GET /api/subscriptions/org/:id` answered any organisation's billing record —
+plan, seat count, price, currency, renewal date — to any signed-in account, and
+`GET /api/subscriptions/` answered every subscription on the platform in one
+list. They now carry `requireOrgAccess` and `requirePlatformStaff`, the two
+gates already used elsewhere in the same file.

package/dist/index.js

@@ -18680,11 +18680,11 @@ async function requireUserFieldWrite(req, userId, field) {
 }
 
 // src/utils/user-response.util.ts
-function withoutCredentials(user) {
+function publicIdentity(user) {
   if (!user || typeof user !== "object")
     return user;
-  const { password, sid, ...rest } = user;
-  return rest;
+  const { _id, name, email } = user;
+  return { _id, name, email };
 }
 
 // src/controllers/user.controller.ts
@@ -18776,7 +18776,7 @@ function useUserController() {
         return;
       }
       const user = await _getByEmail(email);
-      res.json(withoutCredentials(user));
+      res.json(publicIdentity(user));
       return;
     } catch (error2) {
       import_node_server_utils43.logger.log({ level: "error", message: `${error2.message}` });
@@ -31692,6 +31692,7 @@ function useSubscriptionController() {
       return;
     }
     try {
+      await requireOrgAccess(req, _id);
       const data = await _getByOrgId(_id);
       res.json(data);
       return;
@@ -31718,6 +31719,7 @@ function useSubscriptionController() {
     const page = parseInt(req.query.page ?? "1");
     const status = req.query.status ?? "active";
     try {
+      await requirePlatformStaff(req);
       const data = await _getSubscriptions({ search, page, status });
       res.json(data);
       return;
@@ -86094,7 +86096,7 @@ function useUserControllerV2() {
       return rejectValidation(error, next);
     try {
       const user = await _getUserByEmail(value);
-      res.json(withoutCredentials(user));
+      res.json(publicIdentity(user));
       return;
     } catch (err) {
       import_node_server_utils269.logger.log({ level: "error", message: `${err.message}` });