public-share: положительная ветка getSharePage (возврат санитизированного контента анонимной модели) без тестов #85

New Issue

Ghost · 2026-06-21T02:33:12+03:00

Ghost commented

2026-06-21 02:33:12 +03:00

Найдено в multi-aspect code review всех изменений с коммита 053a9c0d (ветка develop).

Грань: test-coverage / security · Severity: warning
Где: apps/server/src/core/ai-chat/tools/public-share-chat-tools.service.ts:117-154

Проблема
Спека гоняет только негативные ветки getSharePage (вне шары, ограниченный потомок). Положительная ветка — страница принадлежит этой шаре, не удалена, не ограничена, контент возвращается — не исполняется. Именно она security-релевантна: перед отдачей контента анонимной модели обязателен updatePublicAttachments (срез comment-марок, токенизация вложений) и затем jsonToMarkdown. Если санитайзер удалят/переставят, все текущие тесты пройдут (они проверяют, что updatePublicAttachments НЕ вызван), и регрессия с утечкой comment-марок/сырых attachment id уедет в прод.

Предлагаемый фикс
Тест: getShareForPage резолвит THIS-SHARE, findById возвращает живую страницу, hasRestrictedAncestor=false — проверить, что updatePublicAttachments вызван с page и результат {title, markdown}. Второй кейс: findById возвращает soft-deleted страницу — generic-ошибка без фетча контента.

Найдено в multi-aspect code review всех изменений с коммита `053a9c0d` (ветка develop). **Грань:** test-coverage / security · **Severity:** warning **Где:** `apps/server/src/core/ai-chat/tools/public-share-chat-tools.service.ts:117-154` **Проблема** Спека гоняет только негативные ветки getSharePage (вне шары, ограниченный потомок). Положительная ветка — страница принадлежит этой шаре, не удалена, не ограничена, контент возвращается — не исполняется. Именно она security-релевантна: перед отдачей контента анонимной модели обязателен updatePublicAttachments (срез comment-марок, токенизация вложений) и затем jsonToMarkdown. Если санитайзер удалят/переставят, все текущие тесты пройдут (они проверяют, что updatePublicAttachments НЕ вызван), и регрессия с утечкой comment-марок/сырых attachment id уедет в прод. **Предлагаемый фикс** Тест: getShareForPage резолвит THIS-SHARE, findById возвращает живую страницу, hasRestrictedAncestor=false — проверить, что updatePublicAttachments вызван с page и результат {title, markdown}. Второй кейс: findById возвращает soft-deleted страницу — generic-ошибка без фетча контента.

Ghost referenced this issue

2026-06-21 02:33:45 +03:00

test (сквозное): security-нагруженные гейты исполнения и fail-closed ветки системно не покрыты #97

Ghost referenced this issue from a commit

2026-06-21 03:30:31 +03:00

test(public-share): cover getSharePage positive + soft-deleted branches (#85)

~~Ghost referenced this issue 2026-06-21 04:15:11 +03:00~~

test (сквозное): security-нагруженные гейты исполнения и fail-closed ветки системно не покрыты #97

Ghost referenced this issue

2026-06-21 04:19:31 +03:00

fix: review/red-team batch 2 — 30 issues (security, ws, page-templates, html-embed, mcp, tests, docs) #101

Ghost closed this issue