Repository navigation
Проверять permissions в UseCase и удалить клеймы секций - #74
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 31d3dc8d5f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 241c3e3064
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
0 New Issues
0 Fixed Issues
0 Accepted Issues
|
Codex Review: Didn't find any major issues. 🎉 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Что изменено и зачем
/api/v1/claims/sectionsиCommon/Authentication/Sections. Модули остаются группировкой permissions:moduleCode/moduleDisplayName.Permissions.Identity.Roles/UsersиPermissions.Compendium.Factions/Heroes/Units. Строковые коды всех 20 permissions сохранены.permissionиз каталога. МиграцияRemoveNonPermissionClaimsудаляет все остальные типы клеймов у пользователей и ролей; исторические миграции сохранены./connect/authorizeпроверяет доступность пользователя отдельным query: загружаетUserReadDbModelбез ролей и клеймов и вызываетEnsureAvailable(UserReadDbModel)./connect/tokenзагружает актуальные пользовательские и ролевые клеймы. Permissions включены в ID token для UI.PermissionAuthorizationTestsудалены; замечания Roslyn по тестовым коллекциям исправлены.Frontend: tactical-heroes/admin#58.
Удаление константы Section из библиотеки: panixida-dotnet-core/application#23.
Как проверить локально
Нужен Docker. Проверка обновлённой миграции:
Последнее изменение: сборки IntegrationTests и FunctionalTests без предупреждений и ошибок; проверка миграции на PostgreSQL и 4 HTTP-теста GetPermissions прошли. Проверено форматирование. Миграция повторно выполняется в тесте: permissions, включая смешанный регистр/пробелы в типе и значения вне каталога, сохраняются; section, custom и пустые типы удаляются.
Полный CI на предыдущем коммите
241c3e3прошёл, включая Sonar без новых замечаний. На последнем изменении полный набор локально повторно не запускался.Совместимость
permission; откат не восстанавливает удалённые назначения. Миграция переименована изRemoveSectionClaimsвRemoveNonPermissionClaims.sectionCodeзаменён метаданными модуля; административные операции требуют permissions; произвольные типы назначаемых клеймов не поддерживаются.tactical-heroes-serviceиспользуется для introspection; собственные бизнес-права иApiPermissionsему пока не добавляются.