Skip to content

Reject duplicate permission roles per entity - #3883

Draft
Souvik Ghosh (souvikghosh04) with Copilot wants to merge 2 commits into
mainfrom
copilot/update-validate-catch-duplicate-roles
Draft

Souvik Ghosh (souvikghosh04) with Copilot wants to merge 2 commits into
mainfrom
copilot/update-validate-catch-duplicate-roles

Conversation

Copilot AI commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Why make this change?

dab validate accepts repeated role names within an entity’s permissions, allowing ambiguous permission definitions.

What is this change?

  • Validation: Reject duplicate role names case-insensitively during CLI validation and startup, identifying the role and entity.
  • Scope: Preserve role-name reuse across different entities.

How was this tested?

  • Integration Tests
  • Unit Tests

Sample Request(s)

For a User entity with two jerry permission entries:

dab validate --config dab-config.json
Duplicate role 'jerry' defined for entity 'User'. Role names must be unique per entity.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: souvikghosh04 <210500244+souvikghosh04@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix validate to catch roles with the same name Reject duplicate permission roles per entity Oct 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🥕[Bug]: Update validate to catch 2 roles with the same name.

2 participants