Передавать «request» в сервис — плохая практика?

Question

[eugenedrvnk]

Я пытаюсь обработать процесс авторизации в приложении, и в разных контроллерах мне нужно записать идентификатор пользователя в сессию. Обычный способ - сделать что-то вроде этого:

@Get('callback')
  handleCallback(
    @Req() req,
  ) {
    const userId = this.authService.loginUser();
    req.session.userId = userId;
  }

Но мне приходится дублировать этот код во многих контроллерах, поэтому я хочу вынести его в какую-то отдельную часть кода, может быть что-то вроде этого:

@Injectable()
export class SessionService {
  write(req, userId: number) {
    req.session.userId = userId;
  }

  read(req) {
    return req.session.userId;
  }
}

Нормальная ли практика передавать объект запроса непосредственно сервису? Или это должен быть не сервис, а, может быть, хелпер или ещё что-то? Или мне просто нужно продублировать эту логику во всех контроллерах?

Comments

[StiflerProger]

Passportjs сам же навешивает session.user тебе в запрос. Зачем делать это самому?

Answer 1

[denisromanenko]

А вы не можете миддлвар навесить, который будет это делать в любом случае для запроса?

Answer 2

[Grish Poghosyan]

Можно инжектировать объект запроса в конструкторе

import { Inject } from '@nestjs/common';
import { REQUEST } from '@nestjs/core';

@Injectable()
export class SessionService {
  constructor(@Inject(REQUEST) private readonly req: Express.Request) {};

  write(userId: number) {
    this.req.session.userId = userId;
  }

  read() {
    return this.req.session.userId;
  }
}

Но вашу проблему можно легко решить через guard'ы.