Audit of the shipped Chistyy PDF feature against the actual code found three documentation defects and one misplaced gate: - The SSRF write-up understated the hole. assertPublicUrl resolves DNS exactly once, for the initial URL; the in-browser filter never resolves hostnames at all. Any new hostname after the first navigation (redirect, subresource, fetch, ws://) goes unchecked - a DNS rebind is not even required. Corrected in TECHNICAL.md and the design spec. - The prod gate was tied to TOOLBOX_VISIBLE, but /api/pdf sits in PUBLIC_ROUTES and authenticates itself, so the feature goes live the moment browserless and BROWSER_WS_URL appear on prod - before the flag. Gate is now tied to the renderer. - TECHNICAL.md claimed the browserless port is published on neither staging nor prod. It is published on dev/staging (127.0.0.1:3333) and the SSH tunnel depends on it. - AGENTS.md described a src/proxy.ts that does not exist; route protection lives in src/middleware.ts. Also adds a state snapshot (docs/plans) and a "grabli uklada" section to CLAUDE.md covering the non-obvious conventions already enforced in code: the two ToolUsage ids, the vitest include pattern, page.pdf() without a timeout option, context.route not seeing WebSockets, and NEXT_PUBLIC_* being inlined at build time. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Sy7vY7WQ1A3q1MkgsDd8VB
12 KiB
Чистый PDF — инструмент «URL → PDF» в LMS
Дата: 20260706
Статус: дизайн утверждается
Референс: Clean PDF (pdf.brainysnipe.ru) из поста https://t.me/flowing_abyss/94, инструкция https://flowing-abyss.com/Clean-PDF-for-Zotero
Что делаем
Инструмент для студентов школы: превращает любую веб-страницу в чистый PDF — без рекламы, меню и мусора, с нормальной типографикой, оглавлением, форматами A4/Letter и светлой/тёмной темой. Два сценария плюс открытый API:
- Веб-форма в кабинете (
/tools/clean-pdf): вставил URL — скачал PDF. - Zotero: персональный API-ключ + скрипт для плагина Actions & Tags — правый клик на записи → PDF генерируется и прикрепляется к ней.
- Открытый API для продвинутых:
curlс Bearer-ключом, документация на странице инструмента.
Принятые решения
| Вопрос | Решение |
|---|---|
| Архитектура | Всё внутри LMS (подход B): страница и API в Next.js, рендер — отдельный контейнер browserless в том же compose |
| Кому доступ | Любой платный студент: есть CourseEnrollment на курс со slug ≠ FREE_COURSE_SLUG; admin/curator — без ограничений |
| Видимость | Часть Obsidian Toolbox, живёт под существующим флагом TOOLBOX_VISIBLE; включение на проде — отдельное решение позже |
| Лимит | 100 генераций/месяц на студента, env PDF_MONTHLY_LIMIT (без пересборки), плюс burst-лимит 5/мин |
Пайплайн генерации
URL студента
→ Chromium (browserless) загружает страницу (JS отрабатывает)
→ Next.js забирает итоговый HTML
→ Defuddle (npm, через JSDOM) выделяет главный контент
→ чистый HTML-шаблон: типографика, A4/Letter, light/dark, оглавление из заголовков
→ Playwright page.pdf() в том же browserless
→ файл application/pdf в ответ
- Defuddle — библиотека kepano (автор Obsidian), та же, что у референса: https://github.com/kepano/defuddle
- browserless/chromium — отдельный контейнер в docker-compose:
MAX_CONCURRENT_SESSIONS=2, встроенная очередь,mem_limit~1 ГБ, порт наружу не публикуется. - Next.js подключается по WebSocket через
playwright-core(только клиент, Chromium в образ LMS не попадает — образ не растёт). - Таймаут всего пайплайна ~90 сек; понятные JSON-ошибки: URL недоступен, лимит исчерпан, нет платного доступа.
Данные (Prisma)
Новая модель:
model PdfApiKey {
id String @id @default(cuid())
userId String @unique
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
key String @unique // формат sbpdf_<random>, показывается студенту
createdAt DateTime @default(now())
}
- Ключ создаётся лениво при первом заходе на страницу инструмента.
- Перегенерация заменяет ключ (старый перестаёт работать сразу).
- Учёт использований — существующая
ToolUsageсtool = "clean-pdf-generate"; месячный счётчик = COUNT за календарный месяц. Именно отдельный идентификатор:CopyButtonна странице логирует копирования какtool = "clean-pdf", и они не должны тратить лимит генераций. - Проверка платного доступа выполняется на каждый запрос → рефанд или блокировка аккаунта автоматически отключает и веб, и ключ. Источник истины один — БД LMS.
API
GET /api/pdf?url=<...>&format=A4|Letter&theme=light|dark
- Авторизация, два пути: заголовок
Authorization: Bearer sbpdf_...(Zotero, curl) или активная сессия Better Auth (веб-форма — студенту ключ для веба не нужен). - Роут добавляется в
PUBLIC_ROUTESвmiddleware.ts(иначе 307 → /login — известные грабли). - Ответ:
application/pdf+ заголовкиX-Uses-Count/X-Max-Uses(их же показывает Zotero-скрипт). - Ошибки: JSON
{ error }c честными статусами (401 нет ключа/сессии, 403 нет платного доступа, 429 лимит, 422 плохой URL, 504 не отрендерилось).
Страница /tools/clean-pdf
По образцу существующих инструментов тулбокса (реестр TOOLS, ToolCard, страница + клиентская форма):
- Форма: поле URL, выбор формата и темы, кнопка «Скачать PDF», счётчик «использовано N из 100 в этом месяце».
- Блок «Zotero»: персональный ключ (показать/скопировать/перегенерировать), пошаговая инструкция по Actions & Tags, готовый скрипт с уже подставленными адресом школы и ключом студента (адаптация скрипта референса:
SERVICE_URL = https://school.second-brain.ru, endpoint/api/pdf). - Блок «API»: пример curl.
Дизайн — строго ДС-2 (перед вёрсткой перечитать SBT/02-Стандарты/Дизайн-LMS/DESIGN.md).
Безопасность
Сервис скачивает произвольные URL изнутри прод-стенда — главный риск SSRF:
- Только
http://иhttps://. - Перед загрузкой — резолв DNS и блокировка приватных/зарезервированных диапазонов: localhost/127.x, 10.x, 172.16–31.x, 192.168.x, 169.254.x (метаданные облаков), ::1, fc00::/7, плюс docker-хостнеймы стенда (
db,app,browserless). - Внутри browserless — перехват сетевых запросов страницы (
context.route/context.routeWebSocket): блокирует буквальные приватные/зарезервированные IP и хостыlocalhost/*.local/*.internal. Честно про предел этой защиты:assertPublicUrlрезолвит DNS ровно один раз — для исходного URL, а browser-level фильтр хостнеймы не резолвит вовсе. Значит без проверки уходит любой новый хостнейм после первого перехода: цель редиректа, субресурс,fetchиз JS,ws://. DNS-rebind для обхода даже не нужен — достаточно редиректа на внутреннее имя. Остаточный риск закрывается только на сетевом уровне — см. «Деплой». (уточнено 20260912 аудитом кода) - Порт browserless наружу не публикуется, доступен только приложению по внутренней сети compose.
- Куки/учётные данные пользователя на целевую страницу не передаются.
- Санитайз имени файла в
Content-Disposition. - Лимиты: 100/мес (env) + burst 5/мин на пользователя; сверху — очередь browserless (2 конкурентных рендера).
- ⚠️ Follow-up (не блокирует релиз): оба счётчика лимита считают только успешные генерации (
ToolUsageпишется после успешного рендера) — неудачные попытки (таймаут, 5xx с целевого сайта, зависший рендер) лимит не расходуют. Потенциальный ограниченный self-DoS повторными запросами к тяжёлым/неотвечающим URL. Рассмотреть подсчёт попыток, а не только успехов, отдельной задачей.
Деплой
docker-compose.yml(dev) иdocker-compose.prod.yml: сервисbrowserless(образghcr.io/browserless/chromium), внутренняя сеть, лимиты памяти.- env:
BROWSER_WS_URL,PDF_MONTHLY_LIMIT=100, существующийTOOLBOX_VISIBLE. - Prisma-миграция (
PdfApiKey). - Стандартная схема деплоя LMS: сборка на Hetzner →
docker save | ssh | docker loadна Hoster.kz; browserless на Hoster.kz — обычныйdocker pull. - Hot-standby на Hetzner получает тот же compose (репликация БД уже покрывает
PdfApiKeyиToolUsage).
⚠️ Перед тем как поднять контейнер
browserlessна проде, обязательно захардить его сетевой egress — заблокировать169.254.0.0/16(cloud-metadata) и RFC1918 (10.0.0.0/8,172.16.0.0/12,192.168.0.0/16) на уровне хост-файрвола (DOCKER-USER) или выделенной internal-only docker-сети.Гейт привязан к рендереру, а не к флагу (уточнено 20260912):
TOOLBOX_VISIBLEпрячет только страницы/tools/*, а роут/api/pdfлежит вPUBLIC_ROUTESи авторизуется сам — он станет рабочим для любого платного студента в момент появленияbrowserlessиBROWSER_WS_URL, ещё до поднятия флага. Именно это закрывает остаточный SSRF-риск из раздела «Безопасность»: browser-level фильтр его не закрывает, потому что не переразрешает хостнеймы.
Тестирование
- Юнит: SSRF-валидатор (таблица адресов → допуск/блок), пайплайн Defuddle → HTML-шаблон на фикстурах.
- Интеграция на staging: рендер 3–5 реальных статей (лонгрид, статья с картинками, JS-тяжёлая страница), проверка лимитов и обоих путей авторизации.
- Zotero-скрипт — ручная проверка на живой библиотеке.
- Веб-форма — e2e через agent-browser в
--headed(headless режет антиспам — известные грабли).
Вне рамок (YAGNI)
- Отдельный домен
pdf.second-brain.ru— не нужен, всё наschool.second-brain.ru. - Хранение сгенерированных PDF на сервере — файл отдаётся сразу и не сохраняется.
- Пер-курсовый гейтинг, тарифные лимиты, платные квоты — не сейчас.
- Освещение в рассылке/анонсы — после включения
TOOLBOX_VISIBLEна проде, отдельной задачей.