Skip to content
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

Jenkins Plugin not able to retrieve infomation about user behind proxies requiring authentication #143

Merged
merged 7 commits into from Dec 25, 2021
Merged

Jenkins Plugin not able to retrieve infomation about user behind proxies requiring authentication #143

merged 7 commits into from Dec 25, 2021

Conversation

thecooldrop
Copy link
Contributor

Fix the usage of github authentication plugin behind proxies requiring authentication

If we configure the Jenkins server to be used behind a proxy which requires
authentication, then Jenkins authentication plugin becomes unusable. The
reason for that is the fact that configured username and password are not
passed into the httpclient from Jenkins configuration.

To reproduce this error do following:

  • Start Jenkins server behind proxy requiring authentication
  • Configure the gihub authentication plugin
  • Clear your browser cookies
  • Try to login
  • Observe the login looping due to missing access token

This PR fixes the issue.

…g authentication

If we configure the Jenkins server to be used behind a proxy which requires
authentication, then Jenkins authentication plugin becomes unusable. The
reason for that is the fact that configured username and password are not
passed into the httpclient from Jenkins configuration.

To reproduce this error do following:

- Start Jenkins server behind proxy requiring authentication
- Configure the gihub authentication plugin
- Clear your browser cookies
- Try to login
- Observe the login looping due to missing access token

This commit fixes the issue.
@thecooldrop
Copy link
Contributor Author

Please make a new plugin release once this is merged, because our team is blocked in our Jenkins migration by this issue. Thank you in advanced.

Copy link
Member

@basil basil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

@basil basil merged commit 238f375 into jenkinsci:master Dec 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants