Преподаватель может редактировать чужой курс: как исправить права

Типичная причина — endpoint принимает course ID и проверяет только роль teacher, но не связь пользователя с конкретным курсом или tenant. Злоумышленнику достаточно заменить идентификатор в запросе.

Создайте двух преподавателей в разных организациях и попробуйте изменить чужой course ID через API, не ограничиваясь интерфейсом. Зафиксируйте affected rows и audit actor.

Что проверить в первую очередь

Для ситуации «преподаватель может изменить курс другого автора или организации» сначала зафиксируйте один воспроизводимый пример: время, идентификатор объекта, пользователя или операции, входные данные, версию приложения и фактический результат. Отдельно запишите ожидаемое поведение: каждая операция серверно проверяет роль и принадлежность курса текущей организации, а отказ одинаков для чтения, изменения, публикации и удаления. Не меняйте сразу несколько настроек: один контролируемый шаг должен подтверждать или исключать одну гипотезу.

Начинайте с чтения состояния и журналов. До массового перерасчёта, повторной отправки, удаления записей или изменения прав подготовьте резервную копию и способ отката. Секреты, токены, персональные данные и содержимое документов в диагностические выгрузки не включайте.

• Проверьте owner и tenant ID.
• Сверьте роли editor, author и admin.
• Найдите все mutation endpoints.
• Проверьте вложенные lessons и files.

Почему возникает проблема

Видимый симптом обычно появляется в конце цепочки. Первичная ошибка может находиться в API, фоновой задаче, кеше, очереди, базе, правах доступа или внешнем сервисе. Основной риск этого сценария: чужой контент, задания и ученики изменяются без разрешения, а простое скрытие кнопки не предотвращает прямой API-запрос. Поэтому ручная правка итогового статуса часто временно скрывает проблему, но не устраняет причину.

Разбирайте события по хронологии и ищите первую точку расхождения с бизнес-правилом. Учитывайте повтор запроса, задержку события, параллельное выполнение и восстановление после временного отказа: именно в этих условиях проявляются ошибки согласованности и идемпотентности.

• Авторизация проверяется только в UI.
• Repository ищет course по ID без tenant.
• Кеш permissions не включает организацию.
• Bulk endpoint обходится без policy.

Пошаговая диагностика

Создайте безопасный тестовый сценарий, который не затрагивает реальные списания, рассылки и клиентские данные. Назначьте операции единый correlation ID и проследите его через запрос, бизнес-логику, базу, очередь и внешние интеграции. Для каждого этапа сохраните вход, результат, код ответа, длительность и версию записи.

Сравните рабочий и ошибочный случаи по одним полям. Проверьте формат идентификаторов, часовой пояс, порядок событий, права сервисной учётной записи и актуальность конфигурации на каждом экземпляре. Если результат зависит от повтора, задержки или конкретного узла, это важная часть причины, а не случайность.

• Проследите policy decision.
• Проверьте direct object reference.
• Сравните web и mobile API.
• Найдите изменения без audit actor.

Как реализовать объектную авторизацию

Роль отвечает на вопрос что можно делать, а ownership и tenant — над каким объектом; обе проверки обязательны на сервере.

Надёжная реализация хранит бизнес-состояние явно. У каждого значимого действия есть стабильный идентификатор, владелец, версия правила и проверяемый переход статуса. Повторная доставка события не создаёт второе действие, а запоздавший ответ не возвращает объект в невозможное состояние.

• Запросы scoped текущим tenant.
• Policy получает actor и resource.
• Вложенные объекты наследуют границу курса.
• Аудит фиксирует actor и diff.

Как исправить проблему

Разделите исправление кода и восстановление исторических данных. Сначала устраните подтверждённую первопричину, затем повторите исходный сценарий и только после этого готовьте контролируемую коррекцию записей. Для финансовых данных, прав и аудита предпочтительны компенсирующие действия, а не переписывание истории.

Для существующих записей сформируйте dry-run: объект, старое значение, предлагаемое новое значение и основание. Обновление должно быть идемпотентным, ограниченным точной выборкой и создавать отчёт. Любой временный обход ограничьте сроком, пользователями и областью действия.

• Добавьте policy ко всем mutations.
• Используйте tenant-scoped repository.
• Инвалидируйте permission cache.
• Проверьте фоновые и bulk операции.

Безопасный порядок внедрения

• Сохраните конфигурацию, связанные записи и необходимые журналы; отдельно проверьте реальное восстановление резервной копии.
• Воспроизведите проблему на тестовом объекте и сохраните результат до изменения для последующего сравнения.
• Внесите минимальное изменение под контролем версий, опишите причину, ожидаемый эффект и точный откат.
• Проверьте нормальный сценарий, ошибочный ввод, повтор запроса, параллельную операцию и временный отказ зависимости.
• Выпустите изменение на ограниченную долю трафика или один процесс, если архитектура это позволяет.
• После стабильного наблюдения исправьте исторические данные отдельным контролируемым запуском.

Как проверить результат

Один успешный пример недостаточен. Проверьте крайние значения, одновременные действия, перезапуск процесса и восстановление после сбоя сети. Результат подтверждайте не только интерфейсом, но и состоянием базы, очереди, внешнего сервиса и журналом аудита. Особенно важна повторная доставка уже обработанного события.

Критерии приёмки сформулируйте заранее. Каждый пункт должен давать однозначный ответ, а не субъективную оценку. Если тест нельзя автоматизировать, сохраните короткий регрессионный чек-лист с тестовыми данными и ожидаемыми статусами.

• Автор редактирует свой курс.
• Чужой ID даёт нейтральный отказ.
• Admin действует только в своём tenant.
• Вложенный lesson нельзя изменить обходным URL.

Типичные ошибки при исправлении

• Скрывать кнопку вместо server check.
• Доверять tenant ID клиента.
• Разрешать доступ по угадываемому ID.
• Возвращать подробности чужого объекта.

Не отключайте авторизацию, валидацию, TLS, аудит или контроль дублей ради исчезновения симптома. Такое изменение может сделать интерфейс успешным, но увеличить ущерб при следующем сбое. Временная мера должна иметь владельца, наблюдаемость и дату удаления.

Как предотвратить повторение

Профилактика строится вокруг явных контрактов, идемпотентных обработчиков, наблюдаемости и повторяемых релизов. Система должна быстро показывать нарушение правила после обновления, роста нагрузки или отказа интеграции. Ошибку, которая уже привела к потере денег, данных или заявок, закрепите автоматическим тестом.

Мониторьте полный пользовательский путь, а не только доступность серверов. Техническая метрика должна быть связана с бизнес-результатом: заказ завершён, доступ выдан, файл восстановлен, событие обработано один раз, данные изолированы. Уведомление должно содержать контекст для первого решения.

• Denied object access.
• Изменения без actor.
• Cross-tenant запросы.
• Endpoints без policy coverage.

Что подготовить для технического разбора

• Описание ожидаемого и фактического поведения с точной последовательностью действий.
• Время возникновения, идентификатор тестового объекта и версии затронутых компонентов.
• Обезличенные журналы до и после ошибки с единым correlation ID.
• Список последних изменений, выполненных проверок и условий исчезновения симптома.
• Безопасный доступ к тестовой среде либо минимальный пример без паролей, токенов и персональных данных.

Частые вопросы

Достаточно ли UUID?

Нет, сложный ID не является авторизацией.

Кто может редактировать курс?

Только роли, явно связанные с курсом или его организацией по policy.

Что логировать?

Actor, resource, действие и результат без чувствительного содержимого.

Когда нужна помощь специалиста

Если преподаватель меняет чужие курсы, я могу проверить IDOR, policies и tenant-scoping. Для оценки нужны маршруты API и обезличенная схема ролей.