jenkinsci / jenkinsci/active-directory-plugin

[JENKINS-64921] Recursive group lookup strategy does not work for nested groups with a mix of distribution/security groups

Open
#617 0 comments 0 reactions 0 assignees View on GitHub
component:active-directory-plugin enhancement imported-jira-issue priority:minor resolution:unresolved
Dominant language
Java
Stars
53
Forks
115
PR merge metrics
No merged PRs in 30d

Description

The active directory plugin is able to resolve nested groups for:



  • Security groups: For this, you can use Token-Groups group lookup strategy

  • Distribution groups: Both, nested distribution and security groups can be resolved by using the recursive lookup strategy.

The problem is when we have nested groups with a different nature, something like:


  • Security Group A -> Distribution Group A -> Distribution Group B

In the above example, the plugin will only discover Security Group A with both Token-Groups and Recursive Lookup Strategy.

On the other hand in the example below, the plugin will be able to discover: Security Group A, Security Group B and Security Group C with the Token-Group strategy and ALL of them with the recursive strategy (since it considers both security and distribution)


  • Distribution Group A -> Distribution Group B -> Distribution Group C

  • Security Group A -> Security Group B -> Security Group C

I think we should create a new Group Lookup Strategy called "Recursive Full" which will consider all the groups. The current recursive should be called something like "Recursive light"

---
Originally reported by fbelzunc, imported from: Recursive group lookup strategy does not work for nested groups with a mix of distribution/security groups


  • assignee: fbelzunc
  • status: Open
  • priority: Minor
  • component(s): active-directory-plugin
  • resolution: Unresolved
  • votes: 0
  • watchers: 1
  • imported: 2025-12-07

Raw content of original issue

The active directory plugin is able to resolve nested groups for:



  • Security groups: For this, you can use Token-Groups group lookup strategy

  • Distribution groups: Both, nested distribution and security groups can be resolved by using the recursive lookup strategy.

The problem is when we have nested groups with a different nature, something like:


  • Security Group A -> Distribution Group A -> Distribution Group B

In the above example, the plugin will only discover Security Group A with both Token-Groups and Recursive Lookup Strategy.

On the other hand in the example below, the plugin will be able to discover: Security Group A, Security Group B and Security Group C with the Token-Group strategy and ALL of them with the recursive strategy (since it considers both security and distribution)


  • Distribution Group A -> Distribution Group B -> Distribution Group C

  • Security Group A -> Security Group B -> Security Group C

I think we should create a new Group Lookup Strategy called "Recursive Full" which will consider all the groups. The current recursive should be called something like "Recursive light"

Contributor guide

Open the contributing guide

Research direction

The issue names no files or tests. Start by locating the active-directory plugin's Recursive and Token-Groups lookup strategy entry points, then trace how mixed security and distribution groups are handled. Done means a defined strategy distinguishes the existing behavior from the proposed Recursive Full behavior and covers the illustrated nested-group cases.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
authentication, authorization
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.