itlab-internal-services 2.16.3 → 2.16.5

package/dist/classes/fetch-options/liked-by-options.class.d.ts

@@ -1,6 +1,4 @@
-export declare function LikedByOptions<TBase extends new (...args: any[]) => any>(Base: TBase, options?: {
-    requiredPermissions?: string[];
-}): {
+export declare function LikedByOptions<TBase extends new (...args: any[]) => any>(Base: TBase): {
     new (...args: any[]): {
         [x: string]: any;
         includeLikedBy?: boolean;

package/dist/classes/fetch-options/liked-by-options.class.js

@@ -12,16 +12,14 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.LikedByOptions = LikedByOptions;
 const swagger_1 = require("@nestjs/swagger");
 const class_validator_1 = require("class-validator");
-const pipes_1 = require("../../pipes");
 const transform_1 = require("../../transform");
-function LikedByOptions(Base, options) {
+function LikedByOptions(Base) {
     class LikedByOptions extends Base {
     }
     __decorate([
         (0, swagger_1.ApiPropertyOptional)({}),
         (0, transform_1.BooleanTransform)(),
         (0, class_validator_1.IsOptional)(),
-        (0, pipes_1.Restricted)(options === null || options === void 0 ? void 0 : options.requiredPermissions),
         __metadata("design:type", Boolean)
     ], LikedByOptions.prototype, "includeLikedBy", void 0);
     return LikedByOptions;

package/dist/classes/task-manager.class.d.ts

@@ -24,7 +24,7 @@ export declare class TaskManager {
      * @param accountsService Service responsible for fetching users
      * @returns Map of userId -> user
      */
-    static fetchAndMapUsers(userIdGroups: string[][], accountsService: AccountsService): Promise<Map<string, User>>;
+    static fetchAndMapUsers(userIdGroups: string[][], accountsService: AccountsService): Promise<FetchMap<User>>;
     /**
      * Fetches all users for the given ID groups and returns a mapping.
      * Ensures each user is fetched only once.
@@ -32,23 +32,7 @@ export declare class TaskManager {
      * @param accountsService Service responsible for fetching users
      * @returns Map of userId -> user
      */
-    static fetchAndMapAccounts(accountIdGroups: string[][], accountsService: AccountsService): Promise<Map<string, Account>>;
-    /**
-     * Maps an array of IDs to user objects using a userMap.
-     * Filters out any IDs that don’t have a corresponding user.
-     * @param userIds Array of user IDs to map
-     * @param userMap Map of userId -> user object
-     * @returns Array of user objects
-     */
-    static mapIdsToUsers(userIds: string[], userMap: Map<string, User>): User[];
-    /**
-     * Maps an array of IDs to user objects using a userMap.
-     * Filters out any IDs that don’t have a corresponding user.
-     * @param accountIds Array of user IDs to map
-     * @param accountMap Map of userId -> user object
-     * @returns Array of user objects
-     */
-    static mapIdsToAccounts(accountIds: string[], accountMap: Map<string, Account>): Account[];
+    static fetchAndMapAccounts(accountIdGroups: string[][], accountsService: AccountsService): Promise<FetchMap<Account>>;
     /**
      * Adds a new asynchronous task to the queue.
      *
@@ -69,3 +53,23 @@ export declare class TaskManager {
      */
     clear(): void;
 }
+declare class FetchMap<T extends User | Account> {
+    private readonly map;
+    constructor(items?: T[]);
+    /**
+     * Maps a single ID to an user/account object using a user/account Map.
+     * @param id ID of user/account to map
+     * @param map Map of id -> user/account object
+     * @returns user/account object
+     */
+    get(id: string): T;
+    /**
+     * Maps an array of IDs to user/account objects using a user/account Map.
+     * Filters out any IDs that don’t have a corresponding entity.
+     * @param ids Array of user/account IDs to map
+     * @param map Map of id -> user/account object
+     * @returns Array of user/account objects
+     */
+    getMultiple(ids: string[]): T[];
+}
+export {};

package/dist/classes/task-manager.class.js

@@ -31,11 +31,11 @@ class TaskManager {
         // Collect all unique user IDs across all groups
         const uniqueUserIds = new Set(userIdGroups.flat());
         if (uniqueUserIds.size === 0)
-            return new Map();
+            return new FetchMap();
         // Fetch all users once
         const userCollection = await accountsService.getUserCollection(Array.from(uniqueUserIds));
         // Build a map for quick lookup
-        return new Map(userCollection.map((user) => [user.id, user]));
+        return new FetchMap(userCollection);
     }
     /**
      * Fetches all users for the given ID groups and returns a mapping.
@@ -48,31 +48,11 @@ class TaskManager {
         // Collect all unique user IDs across all groups
         const uniqueAccountIds = new Set(accountIdGroups.flat());
         if (uniqueAccountIds.size === 0)
-            return new Map();
+            return new FetchMap();
         // Fetch all users once
         const accountCollection = await accountsService.getAccountCollection(Array.from(uniqueAccountIds));
         // Build a map for quick lookup
-        return new Map(accountCollection.map((account) => [account.id, account]));
-    }
-    /**
-     * Maps an array of IDs to user objects using a userMap.
-     * Filters out any IDs that don’t have a corresponding user.
-     * @param userIds Array of user IDs to map
-     * @param userMap Map of userId -> user object
-     * @returns Array of user objects
-     */
-    static mapIdsToUsers(userIds, userMap) {
-        return userIds.map((userId) => userMap.get(userId)).filter(Boolean);
-    }
-    /**
-     * Maps an array of IDs to user objects using a userMap.
-     * Filters out any IDs that don’t have a corresponding user.
-     * @param accountIds Array of user IDs to map
-     * @param accountMap Map of userId -> user object
-     * @returns Array of user objects
-     */
-    static mapIdsToAccounts(accountIds, accountMap) {
-        return accountIds.map((accountId) => accountMap.get(accountId)).filter(Boolean);
+        return new FetchMap(accountCollection);
     }
     /**
      * Adds a new asynchronous task to the queue.
@@ -101,3 +81,27 @@ class TaskManager {
     }
 }
 exports.TaskManager = TaskManager;
+class FetchMap {
+    constructor(items = []) {
+        this.map = new Map(items.map((item) => [item.id, item]));
+    }
+    /**
+     * Maps a single ID to an user/account object using a user/account Map.
+     * @param id ID of user/account to map
+     * @param map Map of id -> user/account object
+     * @returns user/account object
+     */
+    get(id) {
+        return this.map.get(id);
+    }
+    /**
+     * Maps an array of IDs to user/account objects using a user/account Map.
+     * Filters out any IDs that don’t have a corresponding entity.
+     * @param ids Array of user/account IDs to map
+     * @param map Map of id -> user/account object
+     * @returns Array of user/account objects
+     */
+    getMultiple(ids) {
+        return ids.map((id) => this.map.get(id)).filter(Boolean);
+    }
+}

package/dist/middleware/http-logging.middleware.js

@@ -33,17 +33,7 @@ const HttpLoggingMiddleware = (req, res, next) => {
         next();
         return;
     }
-    // Defer logging until after the response has finished,
-    // ensuring accurate status information
-    res.on('finish', () => {
-        const { statusCode, statusMessage } = res;
-        // Construct a standardized log message
-        const logMessage = `${method} ${url} - ${statusCode} (${statusMessage})`;
-        // Client errors (4xx) are logged at warn level for visibility
-        // All other responses use standard log level
-        const logLevel = statusCode >= 400 && statusCode < 500 ? 'warn' : 'log';
-        logger[logLevel](logMessage);
-    });
+    logger.log(`Handle ${method} ${url}`);
     // Continue to next middleware or request handler
     next();
 };

package/dist/modules/services/base-http.service.js

@@ -49,7 +49,7 @@ class BaseHttpService {
         // Attach request interceptor for consistent logging of all outgoing requests.
         this.client.interceptors.request.use((config) => {
             var _a;
-            this.logger.log(`[${(_a = config.method) === null || _a === void 0 ? void 0 : _a.toUpperCase()}] ${config.url}`);
+            this.logger.log(`${(_a = config.method) === null || _a === void 0 ? void 0 : _a.toUpperCase()} ${config.url}`);
             return config;
         });
     }

package/dist/modules/services/providers/accounts.service.js

@@ -15,9 +15,11 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.AccountsService = void 0;
 const common_1 = require("@nestjs/common");
 const config_1 = require("@nestjs/config");
+const itlab_functions_1 = require("itlab-functions");
 const authentication_1 = require("../../authentication");
 const base_http_service_1 = require("../base-http.service");
 const base_urls_1 = require("../base-urls");
+const chunkSize = 100;
 /**
  * AccountsService
  *
@@ -57,7 +59,11 @@ let AccountsService = class AccountsService extends base_http_service_1.BaseHttp
     async getAccountCollection(accountIds) {
         if (accountIds.length === 0)
             return [];
-        return this.fetchResourceCollection(`internal/accounts`, { params: { ids: accountIds } });
+        const chunkedAccountIds = (0, itlab_functions_1.createChunks)(accountIds, chunkSize);
+        const chunkedAccounts = chunkedAccountIds.map((accountIds) => {
+            return this.fetchResourceCollection(`internal/accounts`, { params: { ids: accountIds } });
+        });
+        return (await Promise.all(chunkedAccounts)).flat();
     }
     /**
      * Retrieves a single user entry derived from an account.
@@ -77,7 +83,11 @@ let AccountsService = class AccountsService extends base_http_service_1.BaseHttp
     async getUserCollection(userIds) {
         if (userIds.length === 0)
             return [];
-        return this.fetchResourceCollection(`internal/users`, { params: { ids: userIds } });
+        const chunkedUserIds = (0, itlab_functions_1.createChunks)(userIds, chunkSize);
+        const chunkedUsers = chunkedUserIds.map((userIds) => {
+            return this.fetchResourceCollection(`internal/users`, { params: { ids: userIds } });
+        });
+        return (await Promise.all(chunkedUsers)).flat();
     }
 };
 exports.AccountsService = AccountsService;

package/dist/modules/services/providers/notification/notification.service.js

@@ -108,7 +108,7 @@ let NotificationService = NotificationService_1 = class NotificationService exte
             this.logger.error('Cannot unschedule notification: Resource type not provided.');
             return;
         }
-        const endpoint = `internal${this.options.resource}/${resourceId}`;
+        const endpoint = `internal/${this.options.resource}/${resourceId}`;
         this.logger.log(`Unscheduling notification for ${this.options.resource} (${resourceId})`);
         this.client
             .delete(endpoint)

package/dist/pipes/restricted-fields.pipe.js

@@ -66,6 +66,8 @@ let RestrictedFieldsPipe = class RestrictedFieldsPipe {
         // Determine the permissions available to this account
         const grantedPerms = (_b = (_a = this.request.account) === null || _a === void 0 ? void 0 : _a.perms) !== null && _b !== void 0 ? _b : [];
         const restrictions = this.reflector.get(RESTRICTED_METADATA_KEY, (_c = metadata.metatype) === null || _c === void 0 ? void 0 : _c.prototype);
+        if (!(0, class_validator_1.isObject)(restrictions))
+            return value;
         for (const [propertyKey, requiredPerms] of Object.entries(restrictions)) {
             if (requiredPerms.length && !(0, itlab_functions_1.hasSomePermission)(requiredPerms, grantedPerms)) {
                 delete value[propertyKey];

package/dist/swagger/remove-public-route-auth.plugin.js

@@ -18,7 +18,6 @@ function removePublicRouteAuth(document) {
                     method.parameters = method.parameters.filter((parameter) => parameter.name !== service_auth_guard_1.SERVICE_AUTH_HEADER_KEY);
                 }
                 delete method.responses['401'];
-                console.log(method);
             }
         });
     });

package/package.json

@@ -4,7 +4,7 @@
     "name": "Timo Scheuermann",
     "email": "timo.scheuermann@sv-informatik.de"
   },
-  "version": "2.16.3",
+  "version": "2.16.5",
   "type": "commonjs",
   "files": [
     "dist"