aep/sdlc-hub
1
0
Fork 0
Code Issues 24 Pull Requests Actions Packages Projects Releases Wiki Activity

Follow-ups from #43 #49

New Issue
Open
opened 2026-10-16 09:12:41 +03:00 by aep · 1 comment
aep commented 2026-10-16 09:12:41 +03:00
Owner
Copy Link

Отложенные находки финального ревью ветки feat/43-bug-and-attachments (PR #47, merge ee1e240).

  • M-1 — гонка уборки files/ с постановкой пакета или повторной загрузкой: окно — миллисекунды раз в сутки, итог — понятный failed «файл удалён». Источник: финальное ревью #43.
  • M-3 — files.size/files.read ловят только FileNotFoundError: PermissionError/IsADirectoryError при claim дают 500 вместо понятного отказа. Источник: финальное ревью #43.
  • M-4 — filename по умолчанию берётся из path без нормализации NFC: один и тот же кириллический файл по NFD- и NFC-пути даст два вложения. Источник: финальное ревью #43.
Отложенные находки финального ревью ветки `feat/43-bug-and-attachments` (PR #47, merge ee1e240). - **M-1** — гонка уборки `files/` с постановкой пакета или повторной загрузкой: окно — миллисекунды раз в сутки, итог — понятный failed «файл удалён». Источник: финальное ревью #43. - **M-3** — `files.size`/`files.read` ловят только `FileNotFoundError`: `PermissionError`/`IsADirectoryError` при claim дают 500 вместо понятного отказа. Источник: финальное ревью #43. - **M-4** — `filename` по умолчанию берётся из `path` без нормализации NFC: один и тот же кириллический файл по NFD- и NFC-пути даст два вложения. Источник: финальное ревью #43.
aep commented 2026-10-26 09:21:31 +03:00
Author
Owner
Copy Link

PR #130 (merge 6f2cf1a), архив #131: закрыты M-3 — 75b1559, e20977e (нечитаемый файл или каталог на месте файла — «файла нет» с предупреждением в логе, без 500) и M-4 — abbcf0c (filename у ensure_attachment и имя по умолчанию в MCP — NFC). M-1 открыт — issue остаётся открытой.

PR #130 (merge 6f2cf1a), архив #131: закрыты **M-3** — 75b1559, e20977e (нечитаемый файл или каталог на месте файла — «файла нет» с предупреждением в логе, без 500) и **M-4** — abbcf0c (filename у ensure_attachment и имя по умолчанию в MCP — NFC). M-1 открыт — issue остаётся открытой.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aep
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aep/sdlc-hub#49
No description provided.
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?