Наличие роли «менеджер» еще не дает право работать с каждым заказом. На сервере нужно проверять принадлежность конкретного объекта компании, отделу или сотруднику.

Ниже — практический порядок проверки. Он помогает сначала локализовать источник сбоя, затем внести минимальное изменение и проверить результат на реальном сценарии.

Как проявляется проблема

Подменив id заказа в URL или запросе API, менеджер открывает и изменяет заказ другого сотрудника либо другой организации.

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

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

Основные причины

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

  • Контроллер проверяет только роль, но не владельца заказа.
  • Запрос получает заказ по id без ограничения organization_id.
  • Интерфейс скрывает кнопку, однако API принимает прямой запрос.
  • Массовое редактирование использует другой обработчик без object-level проверки.

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

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

Как исправить

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

  • Загружайте заказ сразу в разрешенной области пользователя.
  • Вынесите object-level authorization в общую policy или сервис.
  • Возвращайте 404 либо 403 без раскрытия чужих данных.
  • Записывайте в аудит попытки доступа и изменения ответственных полей.

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

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

Чего не стоит делать

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

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

  • Покрывайте права матрицей ролей и объектов.
  • Проверяйте каждый новый endpoint и экспорт на принадлежность tenant.

Что подготовить для разбора

  • Ссылку на проблемную страницу, метод API, задание, отчет или интеграцию.
  • Точное описание ожидаемого и фактического результата без секретных ключей и паролей.
  • Фрагмент лога за нужный период, идентификатор операции и пример входных данных.
  • Список последних изменений и информацию о рабочем окружении.

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

Можно ли исправить проблему без полной переделки?

Чаще всего да. Если сначала найти точку расхождения, достаточно локальной правки в проверке, транзакции, обработчике события, настройке или запросе к данным.

Почему ошибка появляется не у всех?

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

Итог

Правильная защита выполняется на сервере для каждого объекта и не зависит от скрытых кнопок в интерфейсе.

Если самостоятельно локализовать причину не получилось, я могу разобрать логи и код, воспроизвести ошибку, предложить безопасную правку и проверить ее на рабочем сценарии.