[go: up one dir, main page]

Skip to content

Approval rule shows ineligible approvers

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

Problem: Users with Guest role are displayed in merge request approval lists even though they cannot actually view or approve the MR.

Root Cause: The approver list for an approval rule uses the group_users association (source), which includes all group members, not just those with approve_merge_request permission.

Impact: Confusing UX - developers see "approvers" who can't actually approve.

Reproduction Steps

  1. Create an approval rule with group-based approvers (see docs)
  2. Add a Guest user to that group
  3. Create MR targeting the protected branch → Guest appears in approver list
  4. Impersonate Guest → 404 on MR page

Screenshot_2025-08-26_at_16.33.31

Edited by 🤖 GitLab Bot 🤖