@lenne.tech/nest-server 9.0.23 → 9.0.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lenne.tech/nest-server",
-  "version": "9.0.23",
+  "version": "9.0.25",
   "description": "Modern, fast, powerful Node.js web framework in TypeScript based on Nest with a GraphQL API and a connection to MongoDB (or other databases).",
   "keywords": [
     "node",
@@ -84,7 +84,7 @@
     "light-my-request": "5.8.0",
     "lodash": "4.17.21",
     "mongodb": "4.13.0",
-    "mongoose": "6.8.3",
+    "mongoose": "6.8.4",
     "mongoose-gridfs": "1.3.0",
     "multer-gridfs-storage": "5.0.2",
     "multer": "1.4.5-lts.1",
@@ -95,24 +95,24 @@
     "passport-jwt": "4.0.1",
     "reflect-metadata": "0.1.13",
     "rfdc": "1.3.0",
-    "rimraf": "4.0.4",
+    "rimraf": "4.1.1",
     "rxjs": "7.8.0"
   },
   "devDependencies": {
     "@nestjs/testing": "9.2.1",
     "@types/cron": "2.0.0",
     "@types/ejs": "3.1.1",
-    "@types/jest": "29.2.5",
+    "@types/jest": "29.2.6",
     "@types/lodash": "4.14.191",
     "@types/multer": "1.4.7",
     "@types/node": "18.11.18",
     "@types/nodemailer": "6.4.7",
     "@types/passport": "1.0.11",
     "@types/supertest": "2.0.12",
-    "@typescript-eslint/eslint-plugin": "5.48.1",
-    "@typescript-eslint/parser": "5.48.1",
+    "@typescript-eslint/eslint-plugin": "5.48.2",
+    "@typescript-eslint/parser": "5.48.2",
     "coffeescript": "2.7.0",
-    "eslint": "8.31.0",
+    "eslint": "8.32.0",
     "eslint-config-prettier": "8.6.0",
     "find-file-up": "2.0.1",
     "grunt": "1.5.3",

package/src/core/common/args/filter.args.ts

@@ -32,7 +32,7 @@ export class FilterArgs extends PaginationArgs {
   /**
    * Mapping for Subtypes
    */
-  map(
+  override map(
     data: Partial<this> | Record<string, any>,
     options: {
       cloneDeep?: boolean;

package/src/core/common/args/pagination.args.ts

@@ -63,7 +63,7 @@ export class PaginationArgs extends CoreInput {
   /**
    * Mapping for Subtypes
    */
-  map(
+  override map(
     data: Partial<this> | Record<string, any>,
     options: {
       cloneDeep?: boolean;

package/src/core/common/helpers/input.helper.ts

@@ -324,12 +324,12 @@ export function clone(object: any, options?: { proto?: boolean; circles?: boolea
   try {
     return rfdc(config)(object);
   } catch (e) {
-    console.debug(e);
+    console.debug(e, config, object, 'automatic try to use rfdc with circles');
     if (!config.circles) {
       try {
         return rfdc({ ...config, ...{ circles: true } })(object);
       } catch (e) {
-        console.debug(e);
+        console.debug(e, 'rfcd with circles did not work automatic use of _.clone!');
         return _.clone(object);
       }
     } else {

package/src/core/common/inputs/combined-filter.input.ts

@@ -31,7 +31,7 @@ export class CombinedFilterInput extends CoreInput {
   /**
    * Mapping for Subtypes
    */
-  map(
+  override map(
     data: Partial<this> | Record<string, any>,
     options: {
       cloneDeep?: boolean;

package/src/core/common/inputs/core-input.input.ts

@@ -15,7 +15,7 @@ export abstract class CoreInput extends CoreModel {
   /**
    * Map method
    */
-  public map(
+  public override map(
     data: Partial<this> | Record<string, any>,
     options: {
       cloneDeep?: boolean;

package/src/core/common/inputs/filter.input.ts

@@ -35,7 +35,7 @@ export class FilterInput extends CoreInput {
   /**
    * Mapping for Subtypes
    */
-  map(
+  override map(
     data: Partial<this> | Record<string, any>,
     options: {
       cloneDeep?: boolean;

package/src/core/common/models/core-persistence.model.ts

@@ -83,7 +83,7 @@ export abstract class CorePersistenceModel extends CoreModel {
   /**
    * Initialize instance with default values instead of undefined
    */
-  init() {
+  override init() {
     super.init();
     this.createdAt = this.createdAt === undefined ? new Date() : this.createdAt;
     this.labels = this.labels === undefined ? [] : this.labels;

package/src/core/common/services/mailjet.service.ts

@@ -1,4 +1,5 @@
 import { Injectable } from '@nestjs/common';
+import { template } from 'lodash';
 import { ConfigService } from './config.service';
 // eslint-disable-next-line
 const Mailjet = require('node-mailjet');
@@ -81,7 +82,27 @@ export class MailjetService {
         apiSecret: this.configService.getFastButReadOnly('email.mailjet.api_key_private'),
       });
     } catch (e) {
-      throw new Error('Cannot connect to mailjet.');
+      if (
+        this.configService.getFastButReadOnly('email.mailjet.api_key_public') &&
+        this.configService.getFastButReadOnly('email.mailjet.api_key_private')
+      ) {
+        throw new Error('Cannot connect to mailjet.');
+      }
+      console.debug(
+        JSON.stringify(
+          {
+            info: 'Mailjet credentials are missing',
+            'email.mailjet.api_key_public':
+              this.configService.getFastButReadOnly('email.mailjet.api_key_public') || 'missing',
+            'email.mailjet.api_key_private':
+              this.configService.getFastButReadOnly('email.mailjet.api_key_private') || 'missing',
+            templateData: templateData,
+          },
+          null,
+          2
+        )
+      );
+      return;
     }
 
     // Send mail with mailjet

package/src/core/modules/auth/core-auth.model.ts

@@ -23,7 +23,7 @@ export class CoreAuthModel extends CoreModel {
   /**
    * Initialize instance with default values instead of undefined
    */
-  init() {
+  override init() {
     super.init();
     // Nothing more to initialize yet
     return this;
@@ -32,7 +32,7 @@ export class CoreAuthModel extends CoreModel {
   /**
    * Map input
    */
-  map(input) {
+  override map(input) {
     super.map(input);
     // There is nothing to map yet. Non-primitive variables should always be mapped.
     // If something comes up, you can use `mapClasses` / `mapClassesAsync` from ModelHelper.

package/src/core/modules/auth/guards/roles.guard.ts

@@ -23,7 +23,7 @@ export class RolesGuard extends AuthGuard('jwt') {
   /**
    * Handle request
    */
-  handleRequest(err, user, info, context) {
+  override handleRequest(err, user, info, context) {
     // Get roles
     const reflectorRoles = this.reflector.getAll<string[][]>('roles', [context.getHandler(), context.getClass()]);
     const roles: string[] = reflectorRoles[0]

package/src/core/modules/user/core-user.model.ts

@@ -124,7 +124,7 @@ export abstract class CoreUserModel extends CorePersistenceModel {
   /**
    * Initialize instance with default values instead of undefined
    */
-  public init() {
+  public override init() {
     super.init();
     this.roles = this.roles === undefined ? [] : this.roles;
     return this;
@@ -133,7 +133,7 @@ export abstract class CoreUserModel extends CorePersistenceModel {
   /**
    * Map input
    */
-  map(input) {
+  override map(input) {
     super.map(input);
     // There is nothing to map yet. Non-primitive variables should always be mapped.
     // If something comes up, you can use `mapClasses` / `mapClassesAsync` from ModelHelper.

package/src/core/modules/user/core-user.service.ts

@@ -24,8 +24,8 @@ export abstract class CoreUserService<
 > extends CrudService<TUser> {
   protected constructor(
     protected readonly emailService: EmailService,
-    protected readonly mainDbModel: Model<TUser & Document>,
-    protected readonly mainModelConstructor: CoreModelConstructor<TUser>,
+    protected override readonly mainDbModel: Model<TUser & Document>,
+    protected override readonly mainModelConstructor: CoreModelConstructor<TUser>,
     protected readonly configService?: ConfigService
   ) {
     super();
@@ -38,7 +38,7 @@ export abstract class CoreUserService<
   /**
    * Create user
    */
-  async create(input: any, serviceOptions?: ServiceOptions): Promise<TUser> {
+  override async create(input: any, serviceOptions?: ServiceOptions): Promise<TUser> {
     serviceOptions = merge({ prepareInput: { create: true } }, serviceOptions);
     return this.process(
       async (data) => {

package/src/core/modules/user/inputs/core-user-create.input.ts

@@ -13,5 +13,5 @@ import { CoreUserInput } from './core-user.input';
 export abstract class CoreUserCreateInput extends CoreUserInput {
   @Field({ description: 'Email of the user', nullable: false })
   @IsEmail()
-  email: string = undefined;
+  override email: string = undefined;
 }

package/src/server/common/models/persistence.model.ts

@@ -50,7 +50,7 @@ export abstract class PersistenceModel extends CorePersistenceModel {
   /**
    * Initialize instance with default values instead of undefined
    */
-  init() {
+  override init() {
     super.init();
     // Nothing more to initialize yet
     return this;
@@ -59,7 +59,7 @@ export abstract class PersistenceModel extends CorePersistenceModel {
   /**
    * Map input
    */
-  map(input) {
+  override map(input) {
     super.map(input);
     // There is nothing to map yet. Non-primitive variables should always be mapped.
     // If something comes up, you can use `mapClasses` / `mapClassesAsync` from ModelHelper.

package/src/server/common/services/cron-jobs.service.ts

@@ -12,7 +12,7 @@ export class CronJobs extends CoreCronJobs {
   /**
    * Init cron jobs
    */
-  constructor(protected schedulerRegistry: SchedulerRegistry, protected configService: ConfigService) {
+  constructor(protected override schedulerRegistry: SchedulerRegistry, protected configService: ConfigService) {
     super(schedulerRegistry, configService.config.cronJobs, { log: true });
   }
 

package/src/server/modules/auth/auth.model.ts

@@ -25,7 +25,7 @@ export class Auth extends CoreAuthModel {
   /**
    * Initialize instance with default values instead of undefined
    */
-  init() {
+  override init() {
     super.init();
     // Nothing more to initialize yet
     return this;
@@ -34,7 +34,7 @@ export class Auth extends CoreAuthModel {
   /**
    * Map input
    */
-  map(input) {
+  override map(input) {
     super.map(input);
     return mapClasses(input, { user: User }, this);
   }

package/src/server/modules/file/file.service.ts

@@ -8,7 +8,7 @@ import { CoreFileService } from '../../../core/modules/file/core-file.service';
  */
 @Injectable()
 export class FileService extends CoreFileService {
-  constructor(@InjectConnection() protected readonly connection: Connection) {
+  constructor(@InjectConnection() protected override readonly connection: Connection) {
     super(connection);
   }
 

package/src/server/modules/user/user.model.ts

@@ -54,7 +54,7 @@ export class User extends CoreUserModel implements PersistenceModel {
   /**
    * Initialize instance with default values instead of undefined
    */
-  init() {
+  override init() {
     super.init();
     // Nothing more to initialize yet
     return this;
@@ -63,7 +63,7 @@ export class User extends CoreUserModel implements PersistenceModel {
   /**
    * Map input
    */
-  map(input) {
+  override map(input) {
     super.map(input);
     // There is nothing to map yet. Non-primitive variables should always be mapped.
     // If something comes up, you can use `mapClasses` / `mapClassesAsync` from ModelHelper.

package/src/server/modules/user/user.service.ts

@@ -25,10 +25,10 @@ export class UserService extends CoreUserService<User, UserInput, UserCreateInpu
    * Constructor for injecting services
    */
   constructor(
-    protected readonly configService: ConfigService,
-    protected readonly emailService: EmailService,
-    @Inject('USER_CLASS') protected readonly mainModelConstructor: CoreModelConstructor<User>,
-    @InjectModel('User') protected readonly mainDbModel: Model<UserDocument>,
+    protected override readonly configService: ConfigService,
+    protected override readonly emailService: EmailService,
+    @Inject('USER_CLASS') protected override readonly mainModelConstructor: CoreModelConstructor<User>,
+    @InjectModel('User') protected override readonly mainDbModel: Model<UserDocument>,
     @Inject('PUB_SUB') protected readonly pubSub: PubSub
   ) {
     super(emailService, mainDbModel, mainModelConstructor);
@@ -41,7 +41,7 @@ export class UserService extends CoreUserService<User, UserInput, UserCreateInpu
   /**
    * Create new user and send welcome email
    */
-  async create(input: UserCreateInput, serviceOptions?: ServiceOptions): Promise<User> {
+  override async create(input: UserCreateInput, serviceOptions?: ServiceOptions): Promise<User> {
     // Get prepared user
     let user = await super.create(input, serviceOptions);