Telegram-Market/telegram-shop
1
1
Fork 0
Code Issues 40 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Заменить 214+ console.log на структурированное логирование (pino) #58

New Issue
Closed
opened 2026-06-17 19:39:38 +00:00 by NW · 1 comment
NW commented 2026-06-17 19:39:38 +00:00
Owner
Copy Link

Проблема

214+ вызовов console.log/error без уровней, структуры, ротации. Многие с [DEBUG] в продакшене.

Решение

  • Установить pino и pino-pretty
  • Создать src/utils/logger.js — обёртка над pino с уровнями: debug, info, warn, error
  • Заменить все console.log на logger.info/debug
  • Заменить все console.error на logger.error
  • Убрать [DEBUG] префиксы — использовать уровни логирования
  • Настроить level: process.env.LOG_LEVEL || 'info'
  • Никогда не логировать мнемоники, приватные ключи, токены

Приоритет: СРЕДНИЙ

## Проблема 214+ вызовов `console.log/error` без уровней, структуры, ротации. Многие с `[DEBUG]` в продакшене. ## Решение - [ ] Установить `pino` и `pino-pretty` - [ ] Создать `src/utils/logger.js` — обёртка над pino с уровнями: debug, info, warn, error - [ ] Заменить все `console.log` на `logger.info/debug` - [ ] Заменить все `console.error` на `logger.error` - [ ] Убрать `[DEBUG]` префиксы — использовать уровни логирования - [ ] Настроить `level: process.env.LOG_LEVEL || 'info'` - [ ] Никогда не логировать мнемоники, приватные ключи, токены ## Приоритет: СРЕДНИЙ
NW added this to the Phase 2: Architecture & Refactoring milestone 2026-06-17 19:39:38 +00:00
NW added the priority::mediumtype::refactorphase::2 labels 2026-06-17 19:39:38 +00:00
NW commented 2026-06-22 00:43:02 +00:00
Author
Owner
Copy Link

✅ Реализовано

  • Установить pino и pino-pretty
  • Создать src/utils/logger.js — обёртка над pino
  • Заменить все 207 console.log/error/warn на logger.info/debug/error/warn
  • Убрать [DEBUG], [ERROR] префиксы — уровни логирования уже передают смысл
  • Настроить level: process.env.LOG_LEVEL || 'info'
  • Pino redact: никогда не логировать мнемоники, ключи, токены

46 файлов изменено, 207 вызовов заменено

Коммит: ce1b600

## ✅ Реализовано - [x] Установить `pino` и `pino-pretty` - [x] Создать `src/utils/logger.js` — обёртка над pino - [x] Заменить все 207 console.log/error/warn на logger.info/debug/error/warn - [x] Убрать `[DEBUG]`, `[ERROR]` префиксы — уровни логирования уже передают смысл - [x] Настроить `level: process.env.LOG_LEVEL || 'info'` - [x] Pino redact: никогда не логировать мнемоники, ключи, токены 46 файлов изменено, 207 вызовов заменено **Коммит**: ce1b600
NW closed this issue 2026-06-22 00:43:02 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
phase::1
Phase 1: Security
 phase::1
Phase 1
 phase::1
Phase 1
 phase::2
Phase 2: Architecture
 phase::2
Phase 2
 phase::3
Phase 3: Admin API
 phase::3
Phase 3
 phase::4
Phase 4: Web UI
 phase::4
Phase 4
 phase::5
Phase 5: Quality
 phase::5
Phase 5
 priority::critical
Critical priority
 priority::critical
Critical priority
 priority::high
High priority
 priority::high
High priority
 priority::medium
Medium priority
 priority::medium
Medium priority
 type::bug
Bug fix
 type::bug
Bug
 type::feature
New feature
 type::feature
Feature
 type::infra
Infrastructure
 type::infra
Infrastructure
 type::refactor
Code refactoring
 type::refactor
Refactoring
 type::security
Security issue
 type::security
Security
 type::testing
Testing
 type::testing
Testing
No Label phase::2 priority::medium type::refactor
Milestone
No items
No Milestone Phase 2: Architecture & Refactoring
Projects
Clear projects
No project
Assignees
Clear assignees
NW
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Telegram-Market/telegram-shop#58
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?