Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/github-oauth-plugin
  Commit: fcf8b7721a146e9adcc9d8bf68c88ddc5ed05e52
      
https://github.com/jenkinsci/github-oauth-plugin/commit/fcf8b7721a146e9adcc9d8bf68c88ddc5ed05e52
  Author: Sam Palmer <[email protected]>
  Date:   2021-11-26 (Fri, 26 Nov 2021)

  Changed paths:
    M src/main/java/org/jenkinsci/plugins/GithubAuthenticationToken.java
    M src/test/java/org/jenkinsci/plugins/GithubAccessTokenPropertyTest.java

  Log Message:
  -----------
  [JENKINS-34835] Authorities from team slug (#124)

* [JENKINS-34835] Use team slug for authorities if set

PR #116 loads the team based on the slug if it's available but the
authorities list is still using the team name.
This means that you can set the team in the matrix in global security
but you will then be denied access because the authorities list uses the
team name.
You can see this if you go to /whoAmI The team name is shown against the
user.

* [JENKINS-34835] Add slug to teams

The GithubAuthenticationToken constructor now uses the github team slug
if it's set. I've updated the tests so the slug is set here as well.


-- 
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 [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/github-oauth-plugin/push/refs/heads/master/78c9b5-fcf8b7%40github.com.

Reply via email to