Branch: refs/heads/master
Home: https://github.com/jenkinsci/jenkins
Commit: 9735043b6192df3ba37a5a30d146fb807c3fc9ef
https://github.com/jenkinsci/jenkins/commit/9735043b6192df3ba37a5a30d146fb807c3fc9ef
Author: Wadeck Follonier <wad...@users.noreply.github.com>
Date: 2017-11-12 (Sun, 12 Nov 2017)
Changed paths:
M core/src/main/java/jenkins/security/LastGrantedAuthoritiesProperty.java
R
test/src/test/java/jenkins/security/LastGrantedAuthoritiesPropertyTest.groovy
A
test/src/test/java/jenkins/security/LastGrantedAuthoritiesPropertyTest.java
Log Message:
-----------
[JENKINS-47768] - Avoid having "authenticated" twice in the group membership
of a user (LastGrantedAuthorities) (#3123)
* Avoid having "authenticated" twice in the group membership of a user
- this occur when the SecurityRealm potentially already grants that role (like
in github-oauth-plugin)
* - changed as requested by Oleg
- the list has a maximum of roles.length and in reality it's either
roles.length or (roles.length-1), so the maximum is ok
* - fix problem of missing the "authenticated" authority
* - convert the Groovy script to a Java version
- the Groovy test was not run by default (IIUC Groovy scripts are not compiled
if placed in java src folder)
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to jenkinsci-commits+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
