LMS/docs/16-stage6-report.md

16. Отчёт этапа 6: проверка, hardening и передача

Дата: 2026-10-27. Ветка: feature/groups-courses. Статус: этап 6 закрыт. Остаточные риски перечислены в разделе 4 и не блокируют передачу.

1. Итоговый объём этапов 0–6

Этап	Что реализовано	Отчёт
0	Схема и контракты: identity V4–V10 (V10 — аудит, добавлена на этапе 6), academic V6–V12, progress V13; preflight-скрипты; ADR; контракт профиля студента курса	10
1	Роли ADMIN/TEACHER/STUDENT, группы-когорты и парсер имени, статусы студентов и аккаунтов, membership_history, импорт v2	11
2	Курсы, course_no для браузерных URL, история преподавателей и групп, bootstrap курса C, привязка лабораторных к курсу, копирование лабораторных, админ-UI	12
3	Аккаунты преподавателей, курсовая область видимости, внутренние API identity с LMS_INTERNAL_TOKEN, кабинет преподавателя, профиль студента курса и скачивание всех попыток	13
4	Повышение (dry-run/apply/rollback, планировщик 31 августа МСК), академотпуск, отчисление, выпуск, архив/восстановление, усиленное удаление, блокировка входа	14
5	Курсовые материалы /learn/course/{course_no}/..., публичные глобальные материалы ALL, скрытые материалы для преподавателя, перепроверка статуса web-сессии	15
6	Права преподавателя на запись (лабораторные, ведомость, группы, материалы), центральный аудит identity.audit_events, CI merge gate, hardening и deploy-runbook	этот отчёт

Ключевые инварианты, действующие после этапов 0–6:

• все лабораторные принадлежат курсу; существующие лабораторные, главы и активные группы перенесены в курс C;
• API работает с UUID курса, браузерные URL — с course_no;
• преподаватель видит только активные назначенные курсы; студент — только курсы активной группы; админ — все курсы;
• студент видит только опубликованные оценки и комментарии;
• активы наследуют видимость статьи → раздела → главы: курсовые требуют доступа к курсу, глобальные — аудитории, а старые файлы без привязки остаются публичными (см. раздел 4);
• преподаватель своего курса может писать: лабораторные/уровни/варианты, ведомость, привязка групп и авторские материалы курса; чужие курс, лаба, студент и группа отклоняются на сервисах;
• вход закрыт для отчисленных, выпущенных и архивных студентов, а также для архивных аккаунтов; web дополнительно перепроверяет статус;
• повышение и жизненный цикл пишут историю; применение повышения идемпотентно;
• чувствительные действия пишутся в центральный identity.audit_events, админ смотрит журнал на /admin/audit;
• CI на каждый push/MR прогоняет test ktlintCheck detekt.

2. Проверка

2.1. Сборка, тесты и линтеры

Выполнено 2026-10-27:

./gradlew --no-daemon test ktlintCheck detekt

Результат: BUILD SUCCESSFUL, код возврата 0, все модули зелёные.

Для получения актуальных счётчиков тесты запущены принудительно:

./gradlew test --rerun-tasks

Модуль	Тестов	Failures	Errors	Skipped
identity-service	72	0	0	0
academic-service	88	0	0	0
progress-service	33	0	0	0
web	69	0	0	0
markdown	5	0	0	0
Итого	267	0	0	0

ktlintCheck и detekt прошли без замечаний. Отдельных unit-тестов у gateway и common-http в текущем дереве нет; маршрутизация gateway проверена конфигурацией и security-матрицей ниже. Тот же набор команд выполняет .gitlab-ci.yml.

2.2. Upgrade-тест миграций

Проверка выполнена на одноразовом PostgreSQL 18 (Testcontainers/контейнер, не production-БД):

1. Развёрнута схема master: identity V1–V3, academic V1–V5, progress V1–V9, V11, V12.
2. Загружены синтетические данные: администратор и два студента, две группы (ПИ-11БО, ПИ-21БО), зачисления, семестр, лабораторная, глава/раздел/ статья/актив, оценка, комментарий и попытка сдачи.
3. Поверх master применены миграции ветки:
   • identity V4–V10;
   • academic V6–V12;
   • progress V13.
4. Выполнены postflight-запросы и повторное применение V13.

Результат: успешно, без ошибок.

• identity: группы разобраны в stream/course/number/degree/enrollment_year (ПИ-11БО → поток ПИ, курс 1, номер 1, БО, год набора 2026; ПИ-21БО → курс 2, год 2025); статус ACTIVE; созданы promotion_runs, promotion_run_items; зачисления перенесены в membership_history действием ENROLL; добавлен accounts.status; V10 создаёт identity.audit_events без внешних ключей и индексы по occurred_at/action.
• academic: добавлены labs.course_id, chapters.course_id/audience, индексы labs_number_course_uidx, chapters_slug_global_uidx, chapters_slug_course_uidx, article_assets_article_idx, courses_title_lower_uidx. V12__asset_links_backfill.sql восстанавливает связи article_assets из markdown-ссылок /api/v1/content/assets/{name}; файлы без ссылок остаются несвязанными. Курс C миграциями не создаётся — его создаёт bootstrap после входа админа, поэтому labs_without_course до bootstrap ожидаемо больше нуля.
• progress: grade_comments.author_account_id и lab_grades.updated_by_account_id созданы как nullable uuid; индекс lab_grades_lab_idx существует и валиден; старый комментарий остался без автора, как и задумано.
• V13 идемпотентна: повторное применение V13 (дважды) завершилось сообщениями already exists, skipping и не изменило схему. Это же проверяет ProfileAuthorsMigrationTest на Testcontainers.
• progress V10 отсутствует: в ветке нет progress-service/.../V10__profile_authors.sql; в истории прогресса применяются V11, V12, затем V13. У academic V10 — это другой файл, V10__course_title_unique.sql (уникальный индекс названия курса).
• Переход forward-only: после V13 откат на сборку без неё не пройдёт валидацию Flyway; out-of-order включать нельзя.

2.3. Security-проверки

Проверено автоматическими тестами (полный прогон 267 тестов) и разбором конфигурации gateway. Запись преподавателя дополнительно покрыта negative-тестами «свой/чужой курс».

Сценарий	Ожидание	Факт
Аноним → курс/материалы курса	нет доступа	401 на /api/v1/courses/{id}/content/tree; /api/v1/courses/by-no/{no} — 401
Аноним → публичные материалы	только глобальные ALL	публичное дерево содержит только глобальные главы с audience = ALL; курсовая глава через /api/v1/content/... — 404
Аноним → актив глобальной ALL-статьи	публично	200
Аноним → курсовой/STUDENTS/TEACHERS актив	404	404, существование файла не раскрывается
Студент своего курса → актив опубликованной статьи	доступ	200; скрытый курсовой актив — 404
Студент/преподаватель чужого курса → курсовой актив	403	403
Преподаватель своего курса → скрытый курсовой актив	доступ	200; админ видит любой актив
Глобальный STUDENTS-актив → преподаватель	403	403; студент получает 200
Legacy-актив без связи со статьёй	публично	200; остаточный риск, раздел 4
Админ → курс, профиль, скачивание	полный доступ	доступ к любому курсу и студенту, скачивание любой попытки курса
Преподаватель своего курса → чтение	курс, группы, профиль, материалы	видит опубликованные и скрытые материалы, всех студентов активной группы курса, неопубликованные оценки и все попытки
Преподаватель своего курса → запись	лабы/уровни/варианты, ведомость, группы, материалы	оценки/комментарии, публикация, уровни/варианты и раздача, защиты (код/reset), привязка групп, авторство материалов; проверки в progress CourseScopeService и academic course/content authoring
Преподаватель чужого курса	403	403 на чтение и запись: материалы, курс по course_no, состав группы, профиль, лабы, оценки, группы
Аудит → админ	чтение журнала	GET /api/v1/audit?limit=&action= и /admin/audit доступны только ADMIN
Аудит → запись из сервисов	только по токену	POST /api/v1/internal/audit с X-Internal-Token; без/неверный токен — 403
Студент → свой профиль	только своё и опубликованное	скрытые оценки/комментарии отсутствуют; чужой studentId — 403
Студент → чужой курс/группа	403	доступ только к курсам активной группы; архивный курс — 403
Чужая сдача / лаба другого курса	403	скачивание отклоняется; своя попытка курса скачивается ZIP
Внутренние API identity	только по токену	неверный/пустой X-Internal-Token — 403, сравнение constant-time; неизвестный id — 404
Gateway	внутренние пути не публикуются	маршрутов на /api/v1/internal/** нет; gateway требует JWT для всего, кроме /actuator, входа/setup/JWKS и публичного /api/v1/content/**
Gateway → заголовки личности	клиент не подменяет	X-Account-Id/Login/Role удаляются и проставляются только после проверки JWT
Web-сессия	быстрый отзыв доступа	статус перепроверяется не реже чем раз в 45 секунд; active=false или 401/403 инвалидируют сессию; недоступность identity не разлогинивает всех

Остаточные риски перечислены в разделе 4: legacy-активы без привязки, 30-дневный JWT для прямого API, ведущие нули номера группы и отсутствие пагинации/N+1 на больших объёмах.

3. Чек-лист деплоя

3.1. Перед деплоем

• Backup обязателен. Выполнить ./deploy/backup.sh backup и ./deploy/backup.sh verify <dir>. ./deploy/deploy.sh up сам вызывает backup.sh backup --if-running; при ошибке backup деплой прерывается. Backup на том же сервере не считается единственной защитой.
• В .env.prod задан LMS_INTERNAL_TOKEN — длинная случайная строка (openssl rand -hex 32), одинаковая для identity, academic и progress. Значение по умолчанию dev-internal-token в production не использовать. Там же задан LMS_DB_PASSWORD; .env.prod не коммитится.
• Проверить, что на сервере нет незакоммиченных изменений и выполнен git pull нужной ветки/тега.
• CI зелёный. .gitlab-ci.yml выполняет ./gradlew --no-daemon test ktlintCheck detekt. Runner должен разрешать privileged-сервисы docker:dind: Testcontainers поднимает PostgreSQL 18 через Docker-in-Docker.

3.2. Порядок выкладки

1. Flyway. ./deploy/deploy.sh up собирает образы и запускает postgres → identity → academic → progress → gateway → web. Миграции применяются автоматически при старте сервисов: identity V4–V10 (V10 — identity.audit_events), academic V6–V12, progress V13. Отдельный ручной запуск Flyway не нужен.
2. Bootstrap курса C. После входа админа открыть /admin/courses. Пока курсов нет, показан блок первичной настройки. Bootstrap идемпотентно создаёт/находит курс «Промышленное программирование на языке C», привязывает активные группы, переносит лабораторные и главы и, если лабораторных без курса не осталось, выполняет ALTER TABLE academic.labs ALTER COLUMN course_id SET NOT NULL. Это post-bootstrap шаг в рантайме, а не Flyway-миграция. Повторный запуск безопасен. Если в БД уже есть курс C, bootstrap не требуется.
3. Postflight. Выполнить psql -h localhost -U lms -d lms -f deploy/preflight/groups-courses-post.sql и проверить:
   • labs_without_course = 0;
   • chapters_without_course = 0 (если все текущие материалы курсовые);
   • ровно один курс C;
   • comments_without_author соответствует старым комментариям;
   • группы без полного разбора отсутствуют;
   • таблица identity.audit_events существует (V10).
4. V10 → V13 — только dev/staging. На production/master записи progress V10__profile_authors.sql нет, ничего удалять не нужно. Не путать с academic V10__course_title_unique.sql — он относится к другой схеме и должен применяться как обычно. Если БД когда-либо применяла старую feature-миграцию V10:
   • сделать backup;
   • проверить deploy/preflight/progress-v10-v13.sql (columns_ok и index_ok должны быть true);
   • удалить только строку истории V10 (success=true) через flyway repair или точечный DELETE из примера в скрипте; объекты схемы не удалять;
   • при success=false сначала убедиться, что частичные объекты откатились;
   • не переставлять V11/V12 и не включать out-of-order. После V13 переход forward-only.

3.3. Smoke после деплоя

• docker compose -f docker-compose.prod.yml --env-file .env.prod ps — все контейнеры running; curl -fsS http://127.0.0.1:8090/actuator/health отдаёт UP.
• Админ: вход, /admin/courses (курс C и счётчики), /admin/groups, /admin/labs, /admin/teachers, /admin/promotion.
• Преподаватель: вход, /teacher, карточка своего курса (имена групп, привязка/отвязка группы по available-groups), ведомость /admin/groups/{groupId}?courseId=... без админских контролов, оценка и публикация, уровень/вариант и раздача, защита (код/reset), профиль студента, скачивание попытки и ZIP группы, авторство материалов /teacher/courses/{id}/content.
• Студент: вход, дашборд, лабораторная, сдача/история, курсовые материалы; видны только опубликованные оценки и комментарии.
• Материалы курса: админ и преподаватель видят скрытые статьи, студент — только опубликованные; чужой курс даёт 403, аноним уходит на /login.
• Активы: картинки курсовой статьи открываются у студента своего курса и преподавателя; чужой курс — 403, скрытая картинка у студента — 404, публичная ALL — без входа; старый файл без привязки — публичен.
• Копирование лабораторной: админ копирует лабораторную в другой курс; создаётся черновик с уровнями и вариантами; оценки, сдачи, защиты и назначения не копируются; занятый номер даёт 409.
• Профиль: преподаватель своего курса видит сводку, комментарии, все попытки и скачивает ZIP; чужой студент/курс — 403.
• Повышение: /admin/promotion строит dry-run за год, данные не меняются; применение — только после проверки плана; откат доступен для завершённого запуска. На production не применять план «на всякий случай».
• Аудит: /admin/audit открывается у админа, показывает события (например, PROFILE_VIEW, SUBMISSION_DOWNLOAD, GRADE_SET, CONTENT_*, TEACHER_*) с фильтром по действию и временем МСК; преподаватель/студент получают 403.
• Проверить логи web, gateway, identity, academic, progress на ERROR/WARN после smoke.

4. Известные пробелы и follow-up

1. Legacy-активы без привязки. Видимость связанных активов реализована: V12 восстанавливает article_assets по markdown-ссылкам, а маршрут /api/v1/content/assets/{name} проверяет курс/аудиторию и публикацию. Остаточный случай — файлы, на которые нет ссылок ни в одной статье: они остаются публичными. Нужна инвентаризация и привязка либо удаление таких файлов; до этого fallback сохраняется, чтобы не ломать старые ссылки. Детали — в 15.
2. Старый формат импорта не поддерживается. Импорт принимает только шаблон v2 (поток/курс/номер/степень/год/ФИО/№/база). Ведущие нули номера группы нормализуются к числу (01 → 1), поэтому ПИ-01БО и ПИ-1БО неразличимы; для разных когорт нужен строковый номер. Ограничение зафиксировано в 11.
3. JWT живёт до 30 дней. Web-сессия перепроверяет статус каждые 45 секунд и закрывает UI при отчислении/архиве, но уже выданный JWT для прямого вызова API действует до истечения. Нужны короткий TTL + refresh, denylist/revocation или проверка статуса на каждом API-вызове.
4. Пагинация и N+1. Журнал аудита ограничен limit (максимум 500), но без offset/keyset; часть списков групп, материалов и ведомости отдаётся целиком. На больших курсах нужны пагинация и пакетные выборки вместо запросов в цикле.

5. Рекомендуемые следующие шаги

Приоритет — сверху вниз:

1. Инвентаризация legacy-активов. Найти файлы без строк в article_assets, привязать их к статьям или удалить; после этого можно убрать публичный fallback для несвязанных файлов.
2. JWT/refresh. Короткий TTL access-токена + refresh, denylist/revocation или проверка статуса на каждом API-вызове; закрыть остаточный риск отчисленного/архивного пользователя с уже выданным токеном.
3. Пагинация и устранение N+1. Keyset/offset для аудита, групп, материалов и ведомости; пакетные выборки профилей, комментариев и сдач.
4. Внешняя интеграция «Контингент»/next.uniyar.ac.ru. Отдельный anti-corruption слой, идемпотентная синхронизация, журнал и отчёт дрифта; внешние идентификаторы добавляются миграцией.

6. Итог

Этап 6 закрыт: права преподавателя на запись, аудит, CI, security-матрица, upgrade-тест миграций, видимость активов и deploy-runbook проверены. Остаточные риски (legacy-активы, 30-дневный JWT, ведущие нули номера группы, пагинация/N+1) задокументированы в разделе 4 и не блокируют передачу.

7. LDD baseline

После этапа 6 текущая функциональность проверена через LDD-контур: реестр docs/evidence/claims.json (31 claim, 19 P0 / 12 P1), gate ./gradlew evidenceCheck, upgrade-скрипт и runtime-smoke полного стека. Результат: 31/31 claims proven, P0 gaps = 0, 267 тестов, upgrade с master-схемы и E2E-поток «админ → группа → курс → преподаватель → лаба» проходят. Подробности и остаточные ограничения — docs/evidence/baseline-2026-10-27.md.