Replace nestjs mailer with nodemailer (#197)

* Replace nestjs mailer with nodemailer

* Fix format date package

* Remove @nestjs-modules/mailer in package.json

apps/api/src/auth/auth.service.ts

@@ -23,6 +23,7 @@ import { OrganizationService } from '../organization/organization.service'
 import { ConfigService } from '@nestjs/config'
 import * as bcrypt from 'bcrypt'
 import { pages } from '@isomera/impl'
+import { HandlebarsTemplate } from '../mailer/types/mailer.types'
 
 @Injectable()
 export class AuthService {
@@ -48,7 +49,7 @@ export class AuthService {
         await this.mailerService.sendEmail(
           user,
           'Email verification',
-          'email-confirmation',
+          HandlebarsTemplate.EMAIL_CONFIRMATION,
           {
             name: user.firstName,
             code: code.code
@@ -143,7 +144,12 @@ export class AuthService {
   }
 
   async sendGreetings(user: UserEntity) {
-    return this.mailerService.sendEmail(user, 'Welcome!', 'welcome', { user })
+    return this.mailerService.sendEmail(
+      user,
+      'Welcome!',
+      HandlebarsTemplate.WELCOME,
+      { user }
+    )
   }
 
   async requestPasswordReset(email: string): Promise<boolean> {
@@ -161,7 +167,7 @@ export class AuthService {
         void this.mailerService.sendEmail(
           user,
           'Password reset code',
-          'password-reset-code',
+          HandlebarsTemplate.PASSWORD_RESET_CODE,
           {
             name: `${user.firstName} ${user.lastName}`,
             code: passwordResetCode,

apps/api/src/mailer/mailer.module.ts

@@ -1,43 +1,9 @@
-import { MailerModule as MailModule } from '@nestjs-modules/mailer'
-import { HandlebarsAdapter } from '@nestjs-modules/mailer/dist/adapters/handlebars.adapter'
 import { Module } from '@nestjs/common'
+import { ConfigModule } from '@nestjs/config'
 import { MailerService } from './mailer.service'
-import { ConfigModule, ConfigService } from '@nestjs/config'
-import { config as dotenvConfig } from 'dotenv'
-
-dotenvConfig({ path: '.env' })
 
 @Module({
-  imports: [
-    MailModule.forRootAsync({
-      imports: [ConfigModule],
-      inject: [ConfigService],
-      useFactory: async (configService: ConfigService) => ({
-        transport: {
-          host: configService.get('MAIL_HOST', 'localhost'),
-          port: configService.get('MAIL_PORT', 1025),
-          secure: false,
-          ...(configService.get('MAIL_USER') && {
-            auth: {
-              type: 'login',
-              user: configService.get('MAIL_USER'),
-              pass: configService.get('MAIL_PASSWORD')
-            }
-          })
-        },
-        defaults: {
-          from: '"No Reply" <[email protected]>'
-        },
-        template: {
-          dir: __dirname + '/templates',
-          adapter: new HandlebarsAdapter(),
-          options: {
-            strict: true
-          }
-        }
-      })
-    })
-  ],
+  imports: [ConfigModule.forRoot()],
   providers: [MailerService],
   exports: [MailerService]
 })

apps/api/src/mailer/mailer.service.spec.ts

@@ -1,25 +1,29 @@
 import { Test, TestingModule } from '@nestjs/testing'
 import { MailerService } from './mailer.service'
-import { MailerService as Mailer } from '@nestjs-modules/mailer'
 import { createMock } from '@golevelup/ts-jest'
+import { ConfigService } from '@nestjs/config'
+
+jest.mock('nodemailer', () => ({
+  createTransport: jest.fn().mockReturnValue({
+    sendMail: jest.fn().mockResolvedValue({ messageId: 'test-message-id' })
+  })
+}))
 
 describe('MailerService', () => {
   let service: MailerService
-  let mockedMailerService: jest.Mocked<Mailer>
+  let configService: ConfigService
 
   beforeEach(async () => {
     const module: TestingModule = await Test.createTestingModule({
       providers: [MailerService]
     })
       .useMocker(token => {
-        if (Object.is(token, Mailer)) {
-          return createMock<Mailer>()
+        if (Object.is(token, ConfigService)) {
+          return createMock<ConfigService>()
         }
       })
       .compile()
 
-    mockedMailerService = module.get<Mailer, jest.Mocked<Mailer>>(Mailer)
-
     service = module.get<MailerService>(MailerService)
   })
 

apps/api/src/mailer/mailer.service.ts

@@ -1,20 +1,86 @@
-import { MailerService as Mailer } from '@nestjs-modules/mailer'
 import { HttpException, HttpStatus, Injectable } from '@nestjs/common'
 import { UserEntity } from '../entities/user.entity'
+import * as nodemailer from 'nodemailer'
+import * as handlebars from 'handlebars'
+import * as fs from 'fs'
+import * as path from 'path'
+import { HandlebarsTemplate } from './types/mailer.types'
+import { ConfigService } from '@nestjs/config'
 
 @Injectable()
 export class MailerService {
-  constructor(private mailerService: Mailer) {}
+  private transporter: nodemailer.Transporter
+  private templateCache: Map<HandlebarsTemplate, handlebars.TemplateDelegate>
+
+  constructor(protected readonly configService: ConfigService) {
+    this.templateCache = new Map()
+
+    const host = this.configService.get<string>('MAIL_HOST', 'localhost')
+    const port = this.configService.get<number>('MAIL_PORT', 1025)
+    const secure = this.configService.get<boolean>('MAILER_SECURE', false)
+    const user = this.configService.get<string>('MAIL_USER')
+    const pass = this.configService.get<string>('MAIL_PASSWORD')
+    const fromName = this.configService.get<string>(
+      'MAIL_FROM_NAME',
+      'No-reply'
+    )
+    const fromAddress = this.configService.get<string>(
+      'MAIL_FROM_ADDRESS',
+      '[email protected]'
+    )
+
+    this.transporter = nodemailer.createTransport(
+      {
+        host,
+        port,
+        secure,
+        auth: {
+          user,
+          pass
+        }
+      },
+      {
+        from: {
+          name: fromName,
+          address: fromAddress
+        }
+      }
+    )
+  }
+
+  private loadTemplate(templateName: HandlebarsTemplate, data: object): string {
+    if (this.templateCache.has(templateName)) {
+      const templateRenderFunction = this.templateCache.get(templateName)
+
+      return templateRenderFunction(data)
+    }
+
+    const templatesFolderPath = path.join(__dirname, './templates')
+    const templatePath = path.join(templatesFolderPath, templateName)
+
+    const templateSource = fs.readFileSync(templatePath, 'utf8')
+
+    const templateRenderFunction = handlebars.compile(templateSource)
+    this.templateCache.set(templateName, templateRenderFunction)
+
+    const finalHtml = templateRenderFunction(data)
+
+    return finalHtml
+  }
+
+  async sendEmail(
+    user: UserEntity,
+    subject: string,
+    template: HandlebarsTemplate,
+    data?: object
+  ) {
+    const html = this.loadTemplate(template, data)
 
-  async sendEmail(user: UserEntity, subject: string, template: string, data) {
     try {
-      return await this.mailerService.sendMail({
+      await this.transporter.sendMail({
         to: user.email,
         subject: subject,
-        template: template,
-        context: {
-          ...data
-        }
+        html: html
       })
     } catch (err) {
       console.error(err)

apps/api/src/mailer/types/mailer.types.ts

@@ -0,0 +1,5 @@
+export enum HandlebarsTemplate {
+  PASSWORD_RESET_CODE = 'password-reset-code',
+  WELCOME = 'welcome',
+  EMAIL_CONFIRMATION = 'email-confirmation'
+}

apps/api/src/user/confirm-code.service.ts

@@ -2,7 +2,6 @@ import { HttpException, HttpStatus, Injectable } from '@nestjs/common'
 import { InjectRepository } from '@nestjs/typeorm'
 import { MoreThan, Repository } from 'typeorm'
 import { generateRandomStringUtil } from '@isomera/utils'
-import { format } from 'date-fns'
 import { ConfirmCodeEntity } from '../entities/confirm-code.entity'
 import { UserEntity } from '../entities/user.entity'
 import { DateTime } from 'luxon'
@@ -38,7 +37,7 @@ export class ConfirmCodeService {
       .createQueryBuilder()
       .where({
         code: code,
-        expiresIn: MoreThan(format(new Date(), 'yyyy-MM-dd HH:MM:ss'))
+        expiresIn: MoreThan(DateTime.now().toFormat('yyyy-MM-dd HH:MM:ss'))
       })
       .andWhere('"userId" = :userId', { userId: user.id })
       .limit(1)

package.json

@@ -16,7 +16,6 @@
   "dependencies": {
     "@compodoc/compodoc": "^1.1.22",
     "@expo/metro-config": "~0.10.7",
-    "@nestjs-modules/mailer": "^1.9.1",
     "@nestjs/common": "^10.0.2",
     "@nestjs/config": "^3.1.1",
     "@nestjs/core": "^10.0.2",