# 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](10-stage0-report.md) | | 1 | Роли `ADMIN/TEACHER/STUDENT`, группы-когорты и парсер имени, статусы студентов и аккаунтов, `membership_history`, импорт v2 | [11](11-stage1-report.md) | | 2 | Курсы, `course_no` для браузерных URL, история преподавателей и групп, bootstrap курса C, привязка лабораторных к курсу, копирование лабораторных, админ-UI | [12](12-stage2-report.md) | | 3 | Аккаунты преподавателей, курсовая область видимости, внутренние API identity с `LMS_INTERNAL_TOKEN`, кабинет преподавателя, профиль студента курса и скачивание всех попыток | [13](13-stage3-report.md) | | 4 | Повышение (dry-run/apply/rollback, планировщик 31 августа МСК), академотпуск, отчисление, выпуск, архив/восстановление, усиленное удаление, блокировка входа | [14](14-stage4-report.md) | | 5 | Курсовые материалы `/learn/course/{course_no}/...`, публичные глобальные материалы `ALL`, скрытые материалы для преподавателя, перепроверка статуса web-сессии | [15](15-stage5-course-materials.md) | | 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: ```bash ./gradlew --no-daemon test ktlintCheck detekt ``` Результат: `BUILD SUCCESSFUL`, код возврата `0`, все модули зелёные. Для получения актуальных счётчиков тесты запущены принудительно: ```bash ./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 `. `./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](15-stage5-course-materials.md). 2. **Старый формат импорта не поддерживается.** Импорт принимает только шаблон v2 (поток/курс/номер/степень/год/ФИО/№/база). Ведущие нули номера группы нормализуются к числу (`01` → `1`), поэтому `ПИ-01БО` и `ПИ-1БО` неразличимы; для разных когорт нужен строковый номер. Ограничение зафиксировано в [11](11-stage1-report.md). 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` (37 claim, 20 P0 / 17 P1), gate `./gradlew evidenceCheck`, upgrade-скрипт, runtime-smoke, mutation testing и браузерный Playwright E2E. Результат: **37/37 claims proven, P0 gaps = 0**, 276 тестов; mutation strength `CourseAccessService` 75%, `PromotionPlanner` 69%; browser E2E 4/4; конкурентные apply/rollback и нагрузочный smoke (p95 0.945 c) зелёные. E2E нашёл и закрыл реальный дефект Jackson 3/Kotlin defaults. Подробности и остаточные ограничения — [`docs/evidence/baseline-2026-10-27.md`](evidence/baseline-2026-10-27.md).