Skip to content

Гард роли в onOpenUsers/onOpenResources #85

Closed

Description

@vladcheck

Что

Навигация ResourceList<->UserList скрывает кнопки для не-админов (visible+managed), но сами обработчики переходят безусловно:

  • ResourceListController.onOpenUsers -> goTo(UserList)
  • UsersListController.onOpenResources -> goTo(ResourceList)

Скрытая кнопка не равна запрету перехода.

Предложение

Проверять isAdmin() в обработчиках (показать ошибку или игнорировать переход для не-админа).

Контекст

Найдено в тестах #79 (PR #84): кейсы nonAdminCannotNavigateTo* пропущены через @disabled до появления гарда.

Part of #29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingjavaPull requests that update java codeready-for-agentFully specified, ready for an AFK agent

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions