fix(tradein): praktika brand only under praktika account — drop kopylov mapping (brand leak) #1758

Merged

lekss361 merged 1 commit from fix/praktika-brand-only-praktika-user into main

2026-06-18 13:36:20 +00:00

lekss361 commented

2026-06-18 13:31:09 +00:00

Owner

Проблема (client-facing бренд-утечка)

Trade-in PDF брендируется по владельцу оценки (created_by → _USERNAME_BRAND, анти-spoofing #7). Маппинг содержал ДВА аккаунта на бренд Практики: praktika И kopylov. → оценки, созданные под kopylov (52 шт), рендерили PDF с лого/брендом ГК Практика. Инцидент: отчёт, ушедший Бруснике, нёс лого Практики (конкурента).

Требование: «Практика показывается только юзеру praktika, остальным — generic».

Решение

_USERNAME_BRAND → только {"praktika": "praktika"} (убран kopylov). Все прочие (kopylov, user2, admin, …) → None → generic дефолт (name «Trade-In Estimator», без лого). Бренд резолвится при генерации PDF (не хранится) → существующие kopylov-оценки сразу станут generic после деплоя, без миграции данных.

roles.yaml не тронут — там kopylov/praktika это РОЛЬ pilot (не бренд). get_brand_for_user, services/brand.py, PDF-рендер — без изменений.

Изменения

app/core/auth.py — _USERNAME_BRAND = {"praktika": "praktika"} + переписан комментарий (причина утечки, #657/#7 ref).
tests/test_me_brand.py — kopylov → None (был → praktika); regression-guard test_user_scope_brand_none_for_kopylov; сохранены praktika → praktika, generic-user → None.

Тесты

Full suite: 1987 passed, 1 skipped, 0 failed. test_rbac/idor/quota/history (юзают kopylov для ролей/quota/owner-scope, НЕ бренд) — зелёные. Ruff clean.

Прод-проверка после деплоя

PDF оценки под kopylov → trade-in-generic-<id>.pdf, без лого Практики; под praktika → trade-in-praktika-<id>.pdf (бренд жив).

## Проблема (client-facing бренд-утечка) Trade-in PDF брендируется по владельцу оценки (`created_by` → `_USERNAME_BRAND`, анти-spoofing #7). Маппинг содержал **ДВА** аккаунта на бренд Практики: `praktika` И `kopylov`. → оценки, созданные под `kopylov` (52 шт), рендерили PDF с лого/брендом ГК Практика. **Инцидент:** отчёт, ушедший Бруснике, нёс лого Практики (конкурента). Требование: «Практика показывается только юзеру praktika, остальным — generic». ## Решение `_USERNAME_BRAND` → только `{"praktika": "praktika"}` (убран `kopylov`). Все прочие (`kopylov`, `user2`, `admin`, …) → `None` → generic дефолт (name «Trade-In Estimator», **без лого**). Бренд резолвится при генерации PDF (не хранится) → **существующие kopylov-оценки сразу станут generic** после деплоя, без миграции данных. `roles.yaml` не тронут — там `kopylov`/`praktika` это РОЛЬ `pilot` (не бренд). `get_brand_for_user`, `services/brand.py`, PDF-рендер — без изменений. ## Изменения - `app/core/auth.py` — `_USERNAME_BRAND = {"praktika": "praktika"}` + переписан комментарий (причина утечки, #657/#7 ref). - `tests/test_me_brand.py` — `kopylov → None` (был `→ praktika`); regression-guard `test_user_scope_brand_none_for_kopylov`; сохранены `praktika → praktika`, generic-user → None. ## Тесты Full suite: **1987 passed, 1 skipped, 0 failed**. test_rbac/idor/quota/history (юзают kopylov для ролей/quota/owner-scope, НЕ бренд) — зелёные. Ruff clean. ## Прод-проверка после деплоя PDF оценки под `kopylov` → `trade-in-generic-<id>.pdf`, без лого Практики; под `praktika` → `trade-in-praktika-<id>.pdf` (бренд жив).

lekss361 added 1 commit 2026-06-18 13:31:10 +00:00