@futdevpro/nts-dynamo 1.6.53 → 1.6.55

package/src/_models/control-models/endpoint-params.control-model.ts

@@ -1,6 +1,6 @@
 
 
-import { NextFunction, Request, Response } from 'express';
+import { Request, Response } from 'express';
 
 import { Dynamo_Array, Dynamo_Error, Dynamo_Log } from '@futdevpro/fsm-dynamo';
 import { DynamoNTS_RouteSecurity } from '../../_enums/route-security.enum';
@@ -134,7 +134,7 @@ export class DynamoNTS_EndpointParams{
       }
 
     } catch (error) {
-      this.error(req, res, error);
+      this.error(req, res, error, issuer);
     }
   }
 
@@ -143,9 +143,11 @@ export class DynamoNTS_EndpointParams{
    * @returns 
    */
   getFullExecution(): (req: Request, res: Response) => Promise<void> {
+    let issuer: string;
+
     return async (req: Request, res: Response) => {
       try {
-        const issuer: string = DynamoNTS_GlobalService?.getAuthService()?.getIssuerFromRequest(req);
+        issuer = DynamoNTS_GlobalService?.getAuthService()?.getIssuerFromRequest(req);
         
         if (this.logRequest) {
           await this.preLog(req, res, issuer);
@@ -172,7 +174,7 @@ export class DynamoNTS_EndpointParams{
           }
         }
       } catch (error) {
-        this.error(req, res, error);
+        this.error(req, res, error, issuer);
       }
     };
   }
@@ -182,7 +184,7 @@ export class DynamoNTS_EndpointParams{
    * @param res 
    * @param error 
    */
-  private error(req: Request, res: Response, error: Error | Dynamo_Error): void {
+  private async error(req: Request, res: Response, error: Error | Dynamo_Error, issuer: string): Promise<void> {
     try {
       let msg: string = `Endpoint catched an error. ${this.name} (${this.endpoint})`;
       msg += this.getPathParamsLogContent(req);
@@ -190,12 +192,17 @@ export class DynamoNTS_EndpointParams{
 
       Dynamo_Log.error(msg);
       console.log(
-        (error as Dynamo_Error)?.flag.includes('DYNAMO') ? 
+        (error as Dynamo_Error)?.flag?.includes('DYNAMO') ? 
           (error as Dynamo_Error).getErrorSimplified() : 
           error, 
         '\n'
       );
 
+      await DynamoNTS_GlobalService.globalErrorHandler?.(error, req, res, issuer).catch(err => {
+        Dynamo_Log.warn('DynamoNTS_GlobalService.globalErrorHandler failed to handle error: ', err);
+        Dynamo_Log.warn('It will proceed as normal.');
+      });
+
       res.status((error as Dynamo_Error)?.___status ?? 501);
       res.send(error);
 
@@ -215,7 +222,8 @@ export class DynamoNTS_EndpointParams{
       }
     } catch (error) {
       console.error(
-        `DYNAMO MULTILEVEL ERROR:DynamoNTS_EndpointParams:error: (${this.name}, ${this.endpoint})` +
+        `\n\nDYNAMO MULTILEVEL ERROR:DynamoNTS_EndpointParams:error: (${this.name}, ${this.endpoint})` +
+        `\n(DYNAMO MULTILEVEL ERROR means, that the ERRORHANDLING is ALSO FAILED, and the error message was not sent.)` + 
         `\nERROR:`, error, '\n'
       );
     }

package/src/_services/base/data.service.ts

@@ -698,7 +698,7 @@ export class DynamoNTS_DataService<T extends Dynamo_Metadata> {
     }
   }
   
-  private _getDefaultErrorSettings(fnName: string, error?: Error | Dynamo_Error) {
+  private _getDefaultErrorSettings(fnName: string, error: Error | Dynamo_Error) {
     return {
       status: (error as Dynamo_Error)?.___status ?? 500,
       message: (error as Error)?.message ?? `${fnName} was UNSUCCESFUL (NTS; ${this.dataParams.dataName})`,

package/src/_services/base/db.service.ts

@@ -100,14 +100,7 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
       }
     });
 
-    if (typeof newData._id !== 'string' || typeof newData._id === 'object') {
-      newData._id = `${newData._id}`;
-      if (typeof data._id !== 'string' || typeof data._id === 'object') {
-        Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.createData)')
-      }
-    }
-    data._id = newData._id;
-    data.__v = newData.__v;
+    data = this.stringifyDataId(newData, 'createData');
 
     return data;
   }
@@ -148,14 +141,7 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
       });
     });
 
-    if (typeof newData._id !== 'string' || typeof newData._id === 'object') {
-      newData._id = `${newData._id}`;
-      if (typeof data._id !== 'string' || typeof data._id === 'object') {
-        Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.modifyData)')
-      }
-    }
-    data._id = newData._id;
-    data.__v = newData.__v;
+    data = this.stringifyDataId(newData, 'modifyData');
 
     return data;
   }
@@ -166,6 +152,15 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
    * @returns data
    */
   async getDataById(id: string): Promise<T> {
+    if (!id) { 
+      throw new Dynamo_Error({
+        ...this._getDefaultErrorSettings('getDataById', new Error(`No ID provided! (NTS DB)`)),
+
+        errorCode: 'NTS-DBS-GI1',
+        message: `get ${this.dataParams.dbName} by ID was unsuccessful (NTS DB)`,
+      });
+    }
+
     let data: T = await this.dataModel.findById(id).then(res => {
       return res?.toObject() as T ?? null;
     }).catch(error => {
@@ -177,12 +172,8 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
       });
     });
 
-    if (data && (typeof data._id !== 'string' || typeof data._id === 'object')) {
-      data._id = `${data._id}`;
-      if (typeof data._id !== 'string' || typeof data._id === 'object') {
-        Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.getDataById)')
-      }
-    }
+    data = this.stringifyDataId(data, 'getDataById');
+
     return data;
   }
 
@@ -217,12 +208,8 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
       });
     });
 
-    if (data && (typeof data._id !== 'string' || typeof data._id === 'object')) {
-      data._id = `${data._id}`;
-      if (typeof data._id !== 'string' || typeof data._id === 'object') {
-        Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.getDataByDependencyId)')
-      }
-    }
+    data = this.stringifyDataId(data, 'getDataByDependencyId');
+
     return data;
   }
 
@@ -259,12 +246,7 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
 
     if (0 < dataList.length) {
       dataList.forEach((data: T) => {
-        if (data && (typeof data._id !== 'string' || typeof data._id === 'object')) {
-          data._id = `${data._id}`;
-          if (typeof data._id !== 'string' || typeof data._id === 'object') {
-            Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.getDataListByDependencyId)')
-          }
-        }
+        data = this.stringifyDataId(data, 'getDataListByDependencyId');
       });
     }
 
@@ -304,12 +286,7 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
 
     if (0 < dataList.length) {
       dataList.forEach((data: T) => {
-        if (data && (typeof data._id !== 'string' || typeof data._id === 'object')) {
-          data._id = `${data._id}`;
-          if (typeof data._id !== 'string' || typeof data._id === 'object') {
-            Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.getDataListByDependencyIds)')
-          }
-        }
+        data = this.stringifyDataId(data, 'getDataListByDependencyIds');
       });
     }
 
@@ -460,25 +437,21 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
       }
     });
 
-    let dataList: T[] = await this.dataModel.find(filter)
-    .then(res => res ?? [])
-    .catch(error => {
-      throw new Dynamo_Error({ 
-        ...this._getDefaultErrorSettings('searchData', error),
-        
-        errorCode: 'NTS-DBS-SD1',
-        message: `search ${this.dataParams.dbName} was unsuccessful (NTS DB)`, 
+    let dataList: T[] = await this.dataModel
+      .find(filter)
+      .then(res => res ?? [])
+      .catch(error => {
+        throw new Dynamo_Error({ 
+          ...this._getDefaultErrorSettings('searchData', error),
+          
+          errorCode: 'NTS-DBS-SD1',
+          message: `search ${this.dataParams.dbName} was unsuccessful (NTS DB)`, 
+        });
       });
-    });
 
     if (0 < dataList.length) {
       dataList.forEach((data: T) => {
-        if (data && (typeof data._id !== 'string' || typeof data._id === 'object')) {
-          data._id = `${data._id}`;
-          if (typeof data._id !== 'string' || typeof data._id === 'object') {
-            Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.searchData)')
-          }
-        }
+        data = this.stringifyDataId(data, 'searchData');
       });
     }
 
@@ -575,12 +548,7 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
 
     if (0 < dataList.length) {
       dataList.forEach((data: T) => {
-        if (data && (typeof data._id !== 'string' || typeof data._id === 'object')) {
-          data._id = `${data._id}`;
-          if (typeof data._id !== 'string' || typeof data._id === 'object') {
-            Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.find)')
-          }
-        }
+        data = this.stringifyDataId(data, 'find');
       });
     }
 
@@ -637,12 +605,7 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
 
     if (0 < dataList.length) {
       dataList.forEach((data: T) => {
-        if (data && (typeof data._id !== 'string' || typeof data._id === 'object')) {
-          data._id = `${data._id}`;
-          if (typeof data._id !== 'string' || typeof data._id === 'object') {
-            Dynamo_Log.error('data._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.findWithPaging)')
-          }
-        }
+        data = this.stringifyDataId(data, 'findWithPaging');
       });
     }
 
@@ -674,12 +637,7 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
       });
     });
 
-    if (typeof newData._id !== 'string' || typeof newData._id === 'object') {
-      newData._id = `${newData._id}`;
-      if (typeof newData._id !== 'string' || typeof newData._id === 'object') {
-        Dynamo_Log.error('newData._id stringifying failed! Please notfiy the developer! (DynamoNTS_DBService.findByIdAndUpdate)')
-      }
-    }
+    newData = this.stringifyDataId(newData, 'findByIdAndUpdate');
 
     return newData;
   }
@@ -823,14 +781,14 @@ export class DynamoNTS_DBService<T extends Dynamo_Metadata> {
   // PRIVATE FUNCTIONS
 
   private stringifyDataId(data: T, fnName: string): T {
-    if (data && (typeof data._id !== 'string' || typeof data._id === 'object')) {
+    if (data?._id && (typeof data._id !== 'string' || typeof data._id === 'object')) {
       data._id = `${data._id}`;
 
       if (typeof data._id !== 'string' || typeof data._id === 'object') {
         data = JSON.parse(JSON.stringify(data));
 
         if (typeof data._id !== 'string' || typeof data._id === 'object') {
-          Dynamo_Log.error(`data._id stringifying failed! Please notfiy the developers! (${fnName})`, new Error());
+          Dynamo_Log.error(`data._id stringifying failed! Please notfiy the DynamoNTS developers! (${fnName})`, new Error());
         }
       }  
     }

package/src/_services/core/email.service.ts

@@ -41,7 +41,7 @@ export class DynamoNTS_EmailService {
     }
   ) {
     try {
-      /* this.serviceName = this.constructor?.name; */
+      this.serviceName = this.constructor?.name;
       // console.log('\n\n\n\n\nNEW CONSTRUCT STARTED', set.email);
       this.senderName = set.senderName;
       this.senderNEmail = `${set.senderName} <${set.email}>`;
@@ -77,16 +77,19 @@ export class DynamoNTS_EmailService {
    * 
    * @param set 
    */
-  public async sendEmail(set: {
-    to: string,
-    subject: string,
-    content?: string,
-    useTemplate?: string,
-    templateProperties?: {
-      [propertyKey: string]: string
-    },
-    attachments?: Attachment[]
-  }): Promise<void> {
+  public async sendEmail(
+    set: {
+      to: string,
+      subject: string,
+      content?: string,
+      useTemplate?: string,
+      templateProperties?: {
+        [propertyKey: string]: string
+      },
+      attachments?: Attachment[]
+    }, 
+    issuer?: string,
+  ): Promise<void> {
     try {
       let content: string;
 
@@ -95,21 +98,25 @@ export class DynamoNTS_EmailService {
       } else {
         if (!set.useTemplate) {
           throw new Dynamo_Error({ 
+            ...this._getDefaultErrorSettings(
+              'sendEmail',
+              new Error(`No content setting passsed to emailSending!`),
+              issuer
+            ),
+
             errorCode: 'NTS-ES4-101',
-            addECToUserMsg: true,
-            message: 'No content setting passsed to emailSending!',
-            userMessage: this.defaultErrorUserMsg,
-            issuerService: this.serviceName,
           });
         }
 
         if (!this.templates[set.useTemplate]) {
           throw new Dynamo_Error({ 
+            ...this._getDefaultErrorSettings(
+              'sendEmail',
+              new Error(`No email template found with this parameter! (${set.useTemplate})`),
+              issuer
+            ),
+
             errorCode: 'NTS-ES4-102',
-            addECToUserMsg: true,
-            message: `No email template found with this parameter! (${set.useTemplate})`,
-            userMessage: this.defaultErrorUserMsg,
-            issuerService: this.serviceName,
           });
         }
 
@@ -124,11 +131,13 @@ export class DynamoNTS_EmailService {
             Dynamo_Log.error('\nDynamoBEEmailService ERROR: INVALID sendEmail settings', set);
             
             throw new Dynamo_Error({ 
+              ...this._getDefaultErrorSettings(
+                'sendEmail', 
+                new Error(`TemplateProperties missing! properties needed: ${props}`), 
+                issuer
+              ),
+
               errorCode: 'NTS-ES4-103',
-              addECToUserMsg: true,
-              message: `TemplateProperties missing! properties needed: ${props}`,
-              userMessage: this.defaultErrorUserMsg,
-              issuerService: this.serviceName,
             });
           }
           this.templatePropertyKeys[set.useTemplate].forEach((propertyKey: string) => {
@@ -159,14 +168,21 @@ export class DynamoNTS_EmailService {
         });
       });
     } catch (error) {
-      throw new Dynamo_Error({ 
-        errorCode: 'NTS-ES4-100',
-        addECToUserMsg: true,
-        message: `SendEmail failed!`,
-        userMessage: this.defaultErrorUserMsg,
-        issuerService: this.serviceName,
-        error
-      });
+      if (error.includes('all recipients were rejected')) {
+        throw new Dynamo_Error({
+          ...this._getDefaultErrorSettings('sendEmail', error, issuer),
+
+          errorCode: 'NTS-ES4-104',
+          userMessage: `Can't send mail to ${set.to}`,
+        })
+      } else {
+        throw new Dynamo_Error({ 
+          ...this._getDefaultErrorSettings('sendEmail', error, issuer),
+
+          errorCode: 'NTS-ES4-100',
+          message: `SendEmail failed!`,
+        });
+      }
     }
   }
 
@@ -240,4 +256,16 @@ export class DynamoNTS_EmailService {
       Dynamo_Log.error(`\nDynamoBEEmailService ERROR, getTemplatePropertyKeys ERROR`, new Error(), '\ntemplate:\n', template);
     }
   }
+  
+  private _getDefaultErrorSettings(fnName: string, error: Error | Dynamo_Error, issuer: string) {
+    return {
+      status: (error as Dynamo_Error)?.___status ?? 500,
+      message: (error as Error)?.message ?? `${fnName} was UNSUCCESFUL (NTS; ${this.serviceName})`,
+      addECToUserMsg: true,
+      userMessage: this.defaultErrorUserMsg,
+      issuer: issuer,
+      issuerService: this.serviceName,
+      error: error,
+    }
+  }
 }

package/src/_services/core/global.service.ts

@@ -1,5 +1,7 @@
 
 
+import { Request, Response } from 'express';
+
 import { Dynamo_Metadata, Dynamo_DataParams, Dynamo_Log, Dynamo_Error } from '@futdevpro/fsm-dynamo';
 import { DynamoNTS_AppParams } from '../../_models/control-models/app-params.control-model';
 
@@ -34,6 +36,9 @@ export class DynamoNTS_GlobalService extends DynamoNTS_SingletonService {
   authService: DynamoNTS_AuthService;
   dbServiceCollection: DynamoNTS_ServiceCollection<DynamoNTS_DBService<any>>; // DynamoNTS_DBServiceCollection;
   emailServiceCollection: DynamoNTS_ServiceCollection<DynamoNTS_EmailService>; // DynamoNTS_EmailServiceCollection;
+
+  private static _globalErrorHandler?: (err: any, req: Request, res: Response, issuer: string) => Promise<void>;
+  static get globalErrorHandler(): (err: any, req: Request, res: Response, issuer: string) => Promise<void> { return this._globalErrorHandler; }
   
   /**
    * You need to setup global Services through this function
@@ -73,6 +78,10 @@ export class DynamoNTS_GlobalService extends DynamoNTS_SingletonService {
     this._params = params;
   }
 
+  static setGlobalErrorHandler(handler?: (err: any, req: Request, res: Response, issuer: string) => Promise<void>): void {
+    this.instance.globalErrorHandler = handler;
+  }
+
   /**
    * 
    * @returns 

package/src/_services/server/app.server.ts

@@ -1,11 +1,14 @@
 
 import Mongoose = require("mongoose");
 import Express = require("express");
+
 import * as Http from 'http';
 import * as Https from 'https';
 import * as FileSystem from 'fs';
 import * as BodyParser from 'body-parser';
 
+import { Request, Response } from 'express';
+
 import { Dynamo_Array, Dynamo_Error, dynamo_error_default, Dynamo_Log, second, wait } from '@futdevpro/fsm-dynamo';
 
 import { DynamoNTS_SingletonService } from '../base/singleton.service';
@@ -241,6 +244,7 @@ export abstract class DynamoNTS_App extends DynamoNTS_SingletonService {
       this.globalService = DynamoNTS_GlobalService.getInstance();
       DynamoNTS_GlobalService.setServices(this.getGlobalServiceCollection());
       DynamoNTS_GlobalService.setParams(this.params);
+      DynamoNTS_GlobalService.setGlobalErrorHandler(this.getGlobalErrorHandler?.());
 
       if (this.getPortSettings) {
         this._ports = this.getPortSettings();
@@ -878,4 +882,9 @@ export abstract class DynamoNTS_App extends DynamoNTS_SingletonService {
    * MISSING Description (TODO)
    */
   postProcess?(): Promise<void>;
+
+  /**
+   * MISSING Description (TODO)
+   */
+  getGlobalErrorHandler?(): (err: any, req: Request, res: Response, issuer: string) => Promise<void>;
 }