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

Удалить fallback ENCRYPTION_KEY из config.js #42

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

Проблема

В src/config/config.js:6 захардкодиован fallback-ключ шифрования:

ENCRYPTION_KEY: process.env.ENCRYPTION_KEY || '9о234893245wer6ga3425670'

Если env var не задана — ВСЕ мнемоники шифруются известным ключом из исходников.

Решение

  • Удалить fallback-значение из config.js
  • Приложение ДОЛЖНО падать при старте без ENCRYPTION_KEY
  • Добавить валидацию: ключ должен быть ≥32 символа
  • Добавить понятное сообщение об ошибке при отсутствии ключа
  • Обновить .env.example с инструкцией по генерации ключа

Файлы

  • src/config/config.js
  • .env.example

Приоритет: КРИТИЧЕСКИЙ

## Проблема В `src/config/config.js:6` захардкодиован fallback-ключ шифрования: ```js ENCRYPTION_KEY: process.env.ENCRYPTION_KEY || '9о234893245wer6ga3425670' ``` Если env var не задана — ВСЕ мнемоники шифруются известным ключом из исходников. ## Решение - [ ] Удалить fallback-значение из config.js - [ ] Приложение ДОЛЖНО падать при старте без ENCRYPTION_KEY - [ ] Добавить валидацию: ключ должен быть ≥32 символа - [ ] Добавить понятное сообщение об ошибке при отсутствии ключа - [ ] Обновить .env.example с инструкцией по генерации ключа ## Файлы - `src/config/config.js` - `.env.example` ## Приоритет: КРИТИЧЕСКИЙ
NW added this to the Phase 1: Security & Secrets Management milestone 2026-06-17 19:39:12 +00:00
NW added the priority::criticaltype::securityphase::1 labels 2026-06-17 19:39:12 +00:00
NW closed this issue 2026-06-17 20:22:51 +00:00
NW commented 2026-06-17 21:04:29 +00:00
Author
Owner
Copy Link

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

  • Удалить fallback-значение из config.js
  • Приложение ДОЛЖНО падать при старте без ENCRYPTION_KEY
  • Добавить валидацию: ключ должен быть ≥32 символа
  • Добавить понятное сообщение об ошибке при отсутствии ключа
  • Обновить .env.example с инструкцией по генерации ключа

Файлы: src/config/config.js, .env.example

## ✅ Реализовано - [x] Удалить fallback-значение из config.js - [x] Приложение ДОЛЖНО падать при старте без ENCRYPTION_KEY - [x] Добавить валидацию: ключ должен быть ≥32 символа - [x] Добавить понятное сообщение об ошибке при отсутствии ключа - [x] Обновить .env.example с инструкцией по генерации ключа **Файлы**: `src/config/config.js`, `.env.example`
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::1 priority::critical type::security
Milestone
No items
No Milestone Phase 1: Security & Secrets Management
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#42
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?