vsezol/space-drive-visualizer

Make logs format identical

Closed this issue · 7 comments

SKY-ALIN commented

Current state:

space-drive-game        | [2024-02-20T10:56:06Z INFO  space_drive_game_server] Server is running on 127.0.0.1:3333
space-drive-visualizer  | [Nest] 1  - 02/20/2024, 10:56:06 AM     LOG [NestFactory] Starting Nest application...
space-drive-visualizer  | [Nest] 1  - 02/20/2024, 10:56:06 AM     LOG [InstanceLoader] AppModule dependencies initialized +26ms
space-drive-visualizer  | [Nest] 1  - 02/20/2024, 10:56:06 AM     LOG [InstanceLoader] VideosModule dependencies initialized +0ms
space-drive-visualizer  | [Nest] 1  - 02/20/2024, 10:56:06 AM     LOG [RoutesResolver] VideosController {/api/videos}: +5ms
space-drive-visualizer  | [Nest] 1  - 02/20/2024, 10:56:06 AM     LOG [RouterExplorer] Mapped {/api/videos/render, POST} route +2ms
space-drive-visualizer  | [Nest] 1  - 02/20/2024, 10:56:06 AM     LOG [NestApplication] Nest application successfully started +1ms
space-drive-visualizer  | [Nest] 1  - 02/20/2024, 10:56:06 AM     LOG 🚀 Application is running on: http://localhost:3000/api
vsezol commented

@SKY-ALIN Что тебе не понравилось в логах?

SKY-ALIN commented

Было бы здорово привести их к общему формату, смотрелось бы очень красиво

vsezol commented

@SKY-ALIN это же разные сервисы на разных языках, не вижу смысла приведения логов к единому формату. Тем более пользователи логи читать не будут

SKY-ALIN commented

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

Игроки перед тем как участвовать будут все дебажить и локально поднимать наши сервисы. Будут видеть )

vsezol commented

@SKY-ALIN не понимаю, какая пользователям разница. Пользователи, которые зачем-то будут разворачивать локально, а не пользоваться просто развернутым тестовым сервером, сами себе усложняют жизнь. Логи не для пользователей, для них есть ошибки и корректные сообщения.

alaev-dev commented

@SKY-ALIN удобно для Elk стека, но пока его нет, зачем усложнять?

SKY-ALIN commented

Elk тут ни при чем. Если не хотите, то ок, можете закрыть ишью. Просто мне показалось, что это было бы элегантно и правильно