@tmlmobilidade/controllers 20260617.1318.33 → 20260617.1731.32

package/dist/agencies/agencies.js

@@ -1,6 +1,7 @@
 /* * */
 import { HTTP_STATUS, HttpException } from '@tmlmobilidade/consts';
 import { agencies } from '@tmlmobilidade/interfaces';
+import { Logger } from '@tmlmobilidade/logger';
 /* * */
 export class AgenciesSharedController {
     //
@@ -20,8 +21,20 @@ export class AgenciesSharedController {
      */
     static async getById(request, reply) {
         const agencyData = await agencies.findById(request.params.id);
-        if (!agencyData)
-            throw new HttpException(HTTP_STATUS.NOT_FOUND, 'Agency not found');
+        if (!agencyData) {
+            const error = new HttpException(HTTP_STATUS.NOT_FOUND, 'Agency not found');
+            Logger.issue({
+                context: {
+                    action: 'getById',
+                    feature: 'agencies',
+                    request,
+                    value: request.params.id,
+                },
+                level: 'error',
+                messageOrError: error,
+            });
+            throw error;
+        }
         reply.send({ data: agencyData, error: null, statusCode: HTTP_STATUS.OK });
     }
 }

package/dist/exporter/exporter.js

@@ -1,6 +1,7 @@
 /* * */
 import { HTTP_STATUS, HttpException } from '@tmlmobilidade/consts';
 import { fileExports, files } from '@tmlmobilidade/interfaces';
+import { Logger } from '@tmlmobilidade/logger';
 /* * */
 export class ExporterSharedController {
     //
@@ -22,16 +23,52 @@ export class ExporterSharedController {
     static async download(request, reply) {
         const { id } = request.params;
         const fileExport = await fileExports.findById(id);
-        if (!fileExport)
-            throw new HttpException(HTTP_STATUS.NOT_FOUND, 'File export not found');
+        if (!fileExport) {
+            const error = new HttpException(HTTP_STATUS.NOT_FOUND, 'File export not found');
+            Logger.issue({
+                context: {
+                    action: 'download',
+                    feature: 'exporter',
+                    request,
+                    value: id,
+                },
+                level: 'error',
+                messageOrError: error,
+            });
+            throw error;
+        }
         // Retrieve file data from database
         const foundFileData = await files.findById(fileExport.file_id);
-        if (!foundFileData)
-            throw new HttpException(HTTP_STATUS.NOT_FOUND, 'File not found');
+        if (!foundFileData) {
+            const error = new HttpException(HTTP_STATUS.NOT_FOUND, 'File not found');
+            Logger.issue({
+                context: {
+                    action: 'download',
+                    feature: 'exporter',
+                    request,
+                    value: id,
+                },
+                level: 'error',
+                messageOrError: error,
+            });
+            throw error;
+        }
         // Stream the file in the given URL to the client
         const storageServiceResponse = await fetch(foundFileData.url);
-        if (!storageServiceResponse.ok || !storageServiceResponse.body)
-            return reply.code(500).send('Could not fetch file.');
+        if (!storageServiceResponse.ok || !storageServiceResponse.body) {
+            const error = new HttpException(HTTP_STATUS.INTERNAL_SERVER_ERROR, 'Could not fetch file');
+            Logger.issue({
+                context: {
+                    action: 'download',
+                    feature: 'exporter',
+                    request,
+                    value: id,
+                },
+                level: 'error',
+                messageOrError: error,
+            });
+            throw error;
+        }
         // Set headers and pipe the response body to the client
         reply.header('Content-Disposition', `attachment; filename="${foundFileData.name}"`);
         reply.header('Content-Type', foundFileData.type);

package/dist/operation/hashed-trips/hashed-trips.js

@@ -20,8 +20,18 @@ export class HashedTripsSharedController {
         // Detect which agency_ids the user has access to,
         // based on their permissions. If none, return an empty array.
         const ridesPermission = PermissionCatalog.get(request.permissions, scope, action);
-        if (!ridesPermission['resources']?.agency_ids?.length)
+        if (!ridesPermission['resources']?.agency_ids?.length) {
+            Logger.issue({
+                context: {
+                    action: 'getBatch',
+                    feature: 'hashedTrips',
+                    request,
+                },
+                level: 'info',
+                messageOrError: 'No agency_ids found in permissions',
+            });
             return reply.send({ data: [], error: null, statusCode: HTTP_STATUS.OK });
+        }
         const allowAllAgencies = ridesPermission['resources'].agency_ids.includes(PermissionCatalog.ALLOW_ALL_FLAG);
         //
         // Get the rides batch using native MongoDB cursor
@@ -48,7 +58,7 @@ export class HashedTripsSharedController {
         //
         // Fetch the rides batch from the database
         const ridesBatch = await rides.aggregate(pipeline);
-        Logger.info(`HashedTripsSharedController.getBatch - ridesBatch count: ${ridesBatch?.length ?? 0}`);
+        Logger.info({ message: `HashedTripsSharedController.getBatch - ridesBatch count: ${ridesBatch?.length ?? 0}` });
         //
         // From the given batch of hashed_trip_ids,
         // fetch the full HashedTrip documents with a single query.

package/dist/operation/lines/batch.js

@@ -73,7 +73,7 @@ export async function getOperationalLinesBatch(query) {
     const pipelineResult = await ridesCollection
         .aggregate(pipeline, { allowDiskUse: true })
         .toArray();
-    Logger.info(`OperationalLinesController.getBatch - pipeline result count: ${pipelineResult?.length ?? 0}`);
+    Logger.info({ message: `OperationalLinesController.getBatch - pipeline result count: ${pipelineResult?.length ?? 0}` });
     //
     // Setup the final Map to keep track of the Operation Lines,
     // using the line_id as the key to avoid duplicates,

package/dist/operation/lines/controller.js

@@ -1,6 +1,7 @@
 /* * */
 import { getOperationalLinesBatch } from './batch.js';
 import { HTTP_STATUS } from '@tmlmobilidade/consts';
+import { Logger } from '@tmlmobilidade/logger';
 import { GetOperationalLinesBatchQuerySchema, PermissionCatalog } from '@tmlmobilidade/types';
 /* * */
 export class OperationalLinesSharedController {
@@ -17,8 +18,18 @@ export class OperationalLinesSharedController {
         // Detect which agency_ids the user has access to,
         // based on their permissions. If none, return an empty array.
         const ridesPermission = PermissionCatalog.get(request.permissions, scope, action);
-        if (!ridesPermission['resources']?.agency_ids?.length)
+        if (!ridesPermission['resources']?.agency_ids?.length) {
+            Logger.issue({
+                context: {
+                    action: 'getBatch',
+                    feature: 'operationalLines',
+                    request,
+                },
+                level: 'info',
+                messageOrError: 'No agency_ids found in permissions',
+            });
             return reply.send({ data: [], error: null, statusCode: HTTP_STATUS.OK });
+        }
         const allowAllAgencies = ridesPermission['resources'].agency_ids.includes(PermissionCatalog.ALLOW_ALL_FLAG);
         //
         // Run the query

package/dist/operation/rides/rides.js

@@ -2,6 +2,7 @@
 import { ridesChangeStream } from './watch.js';
 import { HTTP_STATUS } from '@tmlmobilidade/consts';
 import { rides, ridesBatchAggregationPipeline } from '@tmlmobilidade/interfaces';
+import { Logger } from '@tmlmobilidade/logger';
 import { normalizeRide } from '@tmlmobilidade/normalizers';
 import { GetRidesBatchQuerySchema, PermissionCatalog } from '@tmlmobilidade/types';
 /* * */
@@ -21,8 +22,18 @@ export class RidesSharedController {
         // Detect which agency_ids the user has access to,
         // based on their permissions. If none, return an empty array.
         const ridesPermission = PermissionCatalog.get(request.permissions, scope, action);
-        if (!ridesPermission['resources']?.agency_ids?.length)
+        if (!ridesPermission['resources']?.agency_ids?.length) {
+            Logger.issue({
+                context: {
+                    action: 'getBatch',
+                    feature: 'rides',
+                    request,
+                },
+                level: 'info',
+                messageOrError: 'No agency_ids found in permissions',
+            });
             return reply.send({ data: [], error: null, statusCode: HTTP_STATUS.OK });
+        }
         const allowAllAgencies = ridesPermission['resources'].agency_ids.includes(PermissionCatalog.ALLOW_ALL_FLAG);
         //
         // If search is provided, immediately try to find the ride by ID.
@@ -84,8 +95,19 @@ export class RidesSharedController {
         // Detect which agency_ids the user has access to,
         // based on their permissions. If none, return an empty array.
         const ridesPermission = PermissionCatalog.get(request.permissions, scope, action);
-        if (!ridesPermission['resources']?.agency_ids?.length)
+        if (!ridesPermission['resources']?.agency_ids?.length) {
+            Logger.issue({
+                context: {
+                    action: 'getRideById',
+                    feature: 'rides',
+                    request,
+                    value: request.params['id'],
+                },
+                level: 'info',
+                messageOrError: 'No agency_ids found in permissions',
+            });
             return reply.send({ data: null, error: null, statusCode: HTTP_STATUS.OK });
+        }
         const allowAllAgencies = ridesPermission['resources'].agency_ids.includes(PermissionCatalog.ALLOW_ALL_FLAG);
         //
         // If search is provided, immediately try to find the ride by ID.
@@ -113,8 +135,19 @@ export class RidesSharedController {
         // Detect which agency_ids the user has access to,
         // based on their permissions. If none, return an empty array.
         const ridesPermission = PermissionCatalog.get(request.permissions, scope, action);
-        if (!ridesPermission['resources']?.agency_ids?.length)
+        if (!ridesPermission['resources']?.agency_ids?.length) {
+            Logger.issue({
+                context: {
+                    action: 'getRideByIds',
+                    feature: 'rides',
+                    request,
+                    value: request.query['ids'],
+                },
+                level: 'info',
+                messageOrError: 'No agency_ids found in permissions',
+            });
             return reply.send({ data: null, error: null, statusCode: HTTP_STATUS.OK });
+        }
         const allowAllAgencies = ridesPermission['resources'].agency_ids.includes(PermissionCatalog.ALLOW_ALL_FLAG);
         //
         // If search is provided, immediately try to find the ride by ID.

package/dist/operation/stops/batch.js

@@ -73,7 +73,7 @@ export async function getOperationalStopsBatch(query) {
     const pipelineResult = await ridesCollection
         .aggregate(pipeline, { allowDiskUse: true })
         .toArray();
-    Logger.info(`OperationalStopsController.getBatch - pipeline result count: ${pipelineResult?.length ?? 0}`);
+    Logger.info({ message: `OperationalStopsController.getBatch - pipeline result count: ${pipelineResult?.length ?? 0}` });
     //
     // Setup the final Map to keep track of the Operation Stops,
     // using the stop_id as the key to avoid duplicates,

package/dist/operation/stops/controller.js

@@ -1,6 +1,7 @@
 /* * */
 import { getOperationalStopsBatch } from './batch.js';
 import { HTTP_STATUS } from '@tmlmobilidade/consts';
+import { Logger } from '@tmlmobilidade/logger';
 import { GetOperationalStopsBatchQuerySchema, PermissionCatalog } from '@tmlmobilidade/types';
 /* * */
 export class OperationalStopsSharedController {
@@ -17,8 +18,18 @@ export class OperationalStopsSharedController {
         // Detect which agency_ids the user has access to,
         // based on their permissions. If none, return an empty array.
         const ridesPermission = PermissionCatalog.get(request.permissions, scope, action);
-        if (!ridesPermission['resources']?.agency_ids?.length)
+        if (!ridesPermission['resources']?.agency_ids?.length) {
+            Logger.issue({
+                context: {
+                    action: 'getBatch',
+                    feature: 'operationalStops',
+                    request,
+                },
+                level: 'info',
+                messageOrError: 'No agency_ids found in permissions',
+            });
             return reply.send({ data: [], error: null, statusCode: HTTP_STATUS.OK });
+        }
         const allowAllAgencies = ridesPermission['resources'].agency_ids.includes(PermissionCatalog.ALLOW_ALL_FLAG);
         //
         // Run the query

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@tmlmobilidade/controllers",
-	"version": "20260617.1318.33",
+	"version": "20260617.1731.32",
 	"author": {
 		"email": "iso@tmlmobilidade.pt",
 		"name": "TML-ISO"