22 lines
1.3 KiB
Markdown
22 lines
1.3 KiB
Markdown
# Результаты ревью Опуса — DrHider v0.1
|
|
|
|
29.06.2026
|
|
|
|
## Критичные баги
|
|
|
|
| # | Баг | Серьёзность |
|
|
|---|-----|-------------|
|
|
| 1 | `inn_ul` (10 цифр) перед `inn_fl` (12) — 12-значный матчится как 10 | 🔴 |
|
|
| 2 | Префиксы (ИНН/ОГРН/КПП/БИК/р/с/к/с) теряются при замене | 🔴 |
|
|
| 3 | `passport` матчит любые 10-11 цифр (куски р/с, ОГРН, ИНН) | 🔴 |
|
|
| 4 | `.doc` не обрабатывается — текст пустой, файл уходит битым | 🔴 |
|
|
| 5 | PDF возвращается как `.pdf` но содержит UTF-8 текст → битый PDF | 🔴 |
|
|
| 6 | `_workdir` создаётся и удаляется, но не используется | 🟡 |
|
|
| 7 | `_apply_replacements` без границ слов — подстроки чисел могут зацепиться | 🟡 |
|
|
| 8 | CSV-тип для `inn_fl` ошибочно помечается как `inn_ul` | 🟡 |
|
|
| 9 | `.doc` в accept шаблона, но конвертация не реализована | 🟡 |
|
|
|
|
## Безопасность — ОК
|
|
|
|
`_mapping.clear()` в finally, нет логов с реальными данными, path traversal закрыт, лимиты 500/500MB есть, `esc()` в HTML.
|