jenkinsci / jenkinsci/active-directory-plugin
[JENKINS-63303] AD plugin should use tokenGroups by default
- Dominant language
- Java
- Stars
- 53
- Forks
- 115
- PR merge metrics
- No merged PRs in 30d
Description
The AD plugin uses `AUTO` by default for its groupLookup.
however this has a performance hit and many users do not need this functionality.
Security should be enforced by `SecurityGroups`, thus we should default to only `tokenGroups` rather than auto (or auto should default to tokengroup lookup first - but that may break existing installations).
That way users will not get a slow experience of logging in and authentication lookup that often plagues jenkins with AD setups by default (they will have to opt into it!)
---
Originally reported by
teilo, imported from: AD plugin should use tokenGroups by default
fbelzunc
Raw content of original issue
The AD plugin uses `AUTO` by default for its groupLookup.
however this has a performance hit and many users do not need this functionality.
Security should be enforced by `SecurityGroups`, thus we should default to only `tokenGroups` rather than auto (or auto should default to tokengroup lookup first - but that may break existing installations).
That way users will not get a slow experience of logging in and authentication lookup that often plagues jenkins with AD setups by default (they will have to opt into it!)
Contributor guide
Research direction
Start at the active-directory-plugin entry point that handles the groupLookup setting, and compare the existing AUTO and tokenGroups behavior. Review authentication and configuration tests for default-value and installation-compatibility coverage. Done means the default avoids the unnecessary lookup while preserving an explicit opt-in for users who need it.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- authentication, backend, security
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 32/100