Skip to content

Minimal fix for [JENKINS-54031] GitHub OAuth plugin fails with Jenkins 2.146 #103

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 7, 2018

Conversation

kfarnung
Copy link
Contributor

https://issues.jenkins-ci.org/browse/JENKINS-54031

While testing out #101 it seems like there are a lot of changes to land quickly and safely. This patch aims to make a surgical fix without any cleanup or logic flow changes.

While testing out jenkinsci#101 it seems like there are a lot of changes to land
quickly and safely.  This patch aims to make a surgical fix without any
cleanup or logic flow changes.
@nickmccally
Copy link

Thank you so much. I have compiled and tested - it has fixed the jenkins github oauth issue for my team.

@samrocketman samrocketman self-requested a review November 2, 2018 06:17
@conf
Copy link

conf commented Nov 10, 2018

Any update on this? This is blocking us from upgrade to the most recent Jenkins version :(

@juliangrosshauser
Copy link

This change fixed the permission problems for my team. Thank you!

@stfp
Copy link

stfp commented Nov 21, 2018

@samrocketman just want to bump this back on your radar... thanks!

samrocketman added a commit that referenced this pull request Dec 7, 2018
@samrocketman samrocketman merged commit 89e4c67 into jenkinsci:master Dec 7, 2018
@samrocketman
Copy link
Member

samrocketman commented Dec 7, 2018

Sorry for taking so long. I'm in the midst of moving cross country so my volunteer development time is suffering. In any case, I just released 0.31 which includes this fix.

@stfp
Copy link

stfp commented Dec 7, 2018

Many thanks - we all appreciate your time and effort! And good luck with the move 🚚

@samrocketman
Copy link
Member

There was a lot of git spaghetti in this release 😂 https://github.com/jenkinsci/github-oauth-plugin/network

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants