authorization-token #16

Closed
DmitriyA wants to merge 0 commits from authorization-token into rc
Collaborator

Добавлен JWT токен

Добавлен JWT токен
DmitriyA added 2 commits 2025-04-21 11:22:50 +03:00
VladislavD approved these changes 2025-04-21 11:28:37 +03:00
VladislavD left a comment
Collaborator

approved

approved
@ -10,0 +23,4 @@
# Для development
# COOKIE_SECURE=false
# COOKIE_SAME_SITE=lax
Collaborator

.env спрячь и добавь в .gitignore
для демонстрации веди .env.example в фальш-данными

`.env` спрячь и добавь в `.gitignore` для демонстрации веди `.env.example` в фальш-данными
@ -53,2 +53,4 @@
*.pid.lock
# Игнорировать .env файлы
.env
Collaborator

увидел, хорош

увидел, хорош
Ghost requested changes 2025-04-21 11:58:56 +03:00
@ -16,1 +11,3 @@
}
constructor(private authService: AuthService) { }
@Get('check')

Можешь посмотреть, как устроен MVC шаблон ( View у тебя выступает фронт). Можно разделить архитектурно сервисы к сервисам, контроллеры к контроллерам. На контроллер приходит запрос, далее управление переходит сервису, который в свою очередь лезет в репозиторий (тоже можешь почитать, формально это обёртка над моделями для выполнения каких-то запросов)

Можешь посмотреть, как устроен MVC шаблон ( View у тебя выступает фронт). Можно разделить архитектурно сервисы к сервисам, контроллеры к контроллерам. На контроллер приходит запрос, далее управление переходит сервису, который в свою очередь лезет в репозиторий (тоже можешь почитать, формально это обёртка над моделями для выполнения каких-то запросов)
@ -17,0 +17,4 @@
this.logger.debug(`Check auth request. Headers: ${JSON.stringify(req.headers)}`);
if (!req.user) {
this.logger.warn('Unauthorized access attempt');

Подумай о выводе в отдельные переменные

Подумай о выводе в отдельные переменные
@ -17,0 +25,4 @@
const user = req.user as { userId: number; username: string; login?: string };
const userWithoutPassword = { ...user };
this.logger.log(`User authenticated: ${user.username}`);

Убрать

Убрать
@ -17,0 +32,4 @@
};
}
@Post('login')

Подумай о сваггере

Подумай о сваггере
@ -17,0 +50,4 @@
const { access_token } = await this.authService.login(user);
res.cookie('accecdss_token', access_token, {

ошибочка

ошибочка

camelCase

camelCase
@ -0,0 +7,4 @@
context: ExecutionContext,
): boolean | Promise<boolean> | Observable<boolean> {
const request = context.switchToHttp().getRequest();
return !!request.user; // Проверка что пользователь аутентифицирован

не понятно почему двойное отрицание

не понятно почему двойное отрицание
@ -9,0 +13,4 @@
TypeOrmModule.forFeature([User]),
PassportModule,
JwtModule.register({
secret: process.env.JWT_SECRET || 'your-secret-key',

переделать

переделать
@ -0,0 +16,4 @@
request.user = payload;
return true;
} catch (e) {
return false;

exeption в логах

exeption в логах
@ -0,0 +14,4 @@
},
]),
ignoreExpiration: false,
secretOrKey: process.env.JWT_SECRET || 'your-secret-key',

переделать

переделать
src/main.ts Outdated
@ -13,2 +10,3 @@
// Настройка CORS
app.enableCors({
origin: '*',
origin: [process.env.FRONTEND_URL, "http://dev.msf.enode"], //|| 'http://192.168.2.39:5173', // Точный URL фронтенда

комментарии поправить

комментарии поправить
@ -25,4 +27,4 @@
}
handleConnection(client: Socket) {
this.logger.log(`Client connected: ${client.id}`);

убрать

убрать
@ -43,3 +43,2 @@
try {
// Для запросов с диапазоном - просто возвращаем данные без подписки
if (start && end) {
if (isRangeQuery) {

поясни переменную

поясни переменную
@ -5,4 +5,5 @@ export interface PrometheusMetric {
value: number;
type: string; // Тип метрики ("gauge", "counter", и т. д.)
description?: string; // Описание метрики
status?: string; // Добавляем поле для статуса

?: этачё

?: этачё
@ -68,2 +68,3 @@
type: metricType || 'unknown',
description: metricDescription, // Добавляем описание
description: metricDescription,
status: entry.metric.status || 'green', // Используем статус из Prometheus или 'green' по умолчанию

green в отдельную перменную

green в отдельную перменную
@ -94,2 +95,3 @@
type: metricType || 'unknown',
description: metricDescription, // Добавляем описание
description: metricDescription,
status: entry.metric.status || 'green', // Используем статус из Prometheus или 'green' по умолчанию

серый

серый
DmitriyA added 1 commit 2025-04-21 16:39:24 +03:00
test-org/trust-module-backend/pipeline/pr-rc This commit looks good Details
37690dc79f
corrections after the code review
DmitriyA closed this pull request 2025-04-25 11:53:15 +03:00
All checks were successful
test-org/trust-module-backend/pipeline/pr-rc This commit looks good

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: deployer3000/trust-module-backend#16
There is no content yet.