Как правильно организовать, хочу генерить один токен и использовать в 2 контроллерах?

Question

[Константин]

Код:

import { Controller, Get, Post, Body, Patch, Param, Delete } from '@nestjs/common';
import { MailService } from './mail.service';
import { CreateMailDto } from './dto/mail.dto';

@Controller('mail')
export class MailController {
  constructor(private readonly mailService: MailService) {}

  //const token = Math.floor(1000 + Math.random() * 9000).toString();

  @Post('/send')
  signUp(@Body() dto: CreateMailDto) {
    const url = `${process.env.BASE_URL}:${process.env.PORT}/mail/confirm?token=${token}`;

    return this.mailService.sendUserConfirmation(dto, url);
  }

  @Get(`/confirm:${token}`)
  Confirm() {}
}

Ошибка скрином (подчеркивает):

Answer 1

[Dmitriy Grape]

Не const token = ..., аprivate readonly token = Math.floor(...)

В классах нельзя объявлять поля при помощи let, const, var

Comments

[Константин]

Вы имеете ввиду в конструктор закинуть?
Тогда нижние переменные почему то не видят

@Controller('mail')
export class MailController {
  constructor(private readonly mailService: MailService, private readonly token = Math.floor(1000 + Math.random() * 9000).toString()) {}

  @Post('/send')
  signUp(@Body() dto: CreateMailDto) {
    const url = `${process.env.BASE_URL}:${process.env.PORT}/mail/confirm?token=${token}`;

    return this.mailService.sendUserConfirmation(dto, url);
  }

  @Get(`/confirm:${token}`)
  Confirm() {}
}

[Dmitriy Grape]

Ну, можно и в конструктор, конечно... Я имел в виду просто создать поле класса. А далее через this вы можете ссылаться на него из методов класса.

@Controller('mail')
export class MailController {
  constructor(private readonly mailService: MailService) {}

  private readonly token = Math.floor(1000 + Math.random() * 9000).toString()

  @Post('/send')
  signUp(@Body() dto: CreateMailDto) {
    const url = `${process.env.BASE_URL}:${process.env.PORT}/mail/confirm?token=${this.token}`;

    return this.mailService.sendUserConfirmation(dto, url);
  }
}

А вот в декоратор вы его не засунете, они не имеют доступа к полям класса.

@Get(`/confirm:${this.token}`) // ЭТО ТАК НЕ РАБОТАЕТ!!! 
  Confirm() {}

[Константин]

Rag’n’ Code Man, ага, вот в декораторе, то и проблема... как же отловить переход по ссылке

[Константин]

Аааа, так я могу через параметр отловить же и сравнить

[Dmitriy Grape]

Константин, вам тогда нужно, чтобы для каждого запроса создавался новый инстанс контроллера, чтобы токен был более или менее уникальный (не знаю, с чего вы взяли, что генерировать уникальную последовательность при помощи Math.random() — это хорошая идея)

@Controller({
  path: 'mail',
  scope: Scope.REQUEST,
})

Если делать по-нормальному, то вы должны генерировать какой-нибудь GUID и сохранять его в базе данных с индексом уникальности. Затем отправлять эту ссылку на подтверждение пользователю на почту. Он по ней переходит, вы извлекаете @Param('token') из ссылки, ищете его в БД и если таковой существует, то подтверждаете аккаунт юзера.