Skip to content

question(user-mgmt): should contradictory group combinations be blocked? #1001

Closed

Description

@cristim

Summary (QA row 561 / step 14.11)

When assigning groups to a user, any combination is currently accepted, including contradictory ones (e.g. Viewer plus a higher-permission group). QA suggests adding logic so some group combinations cannot be selected together.

Open question (needs product decision)

In the group-membership-only authz model, permissions are the union of all the user's groups, so "Viewer + Admin" simply grants Admin (the union) rather than being contradictory. Before implementing any restriction we need a decision:

  • Should the UI actively block certain combinations (e.g. warn/prevent Viewer + a higher group), or is union-of-permissions the intended, sufficient behavior?
  • If blocking: what is the exact set of mutually-exclusive groups?

Marking needs-info pending that decision; not auto-implementing.

Activity

  1. added
    triagedItem has been triaged
    status/needs-infoAwaiting clarification from the asker
    type/questionQuestion for the project / asker
    on Jun 5, 2026
  2. added a commit that references this issue on Jun 6, 2026
    53cc077
  3. added
    pr-createdA PR has been opened for this issue (dedup guard for the auto-PR loop)
    pr-mergedThe PR for this issue has been merged
    on Jun 7, 2026
  4. added and removed
    status/needs-infoAwaiting clarification from the asker
    on Jul 17, 2026
  5. cristim commented on Jul 17, 2026

    @cristim
    MemberAuthor

    Product decision (owner): leave as-is. In the group-union authorization model, combining Viewer with a higher-permission group simply unions to the higher permission, so no combination is actually contradictory or a security issue. Blocking specific combinations would add UI friction for no behavioral benefit.

    Closing as wont-do. The distinct validation-UX work (invalid-input feedback) was handled separately by #1405 / PR #1427. QA sheet row 564 will be marked not-a-bug / Ready to Test.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    pr-createdA PR has been opened for this issue (dedup guard for the auto-PR loop)pr-mergedThe PR for this issue has been mergedstatus/wontdoClosed as not-plannedtriagedItem has been triagedtype/questionQuestion for the project / asker

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions