-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add email and display name header injection support #30
base: master
Are you sure you want to change the base?
Conversation
Maven release prepare did not updated the version in the pom.xml
Thank you, it's useful for we to add email directly on header. |
@jenkinsci/code-reviewers |
Why hasn't this been picked up? |
Hello, Thank you for the awesome plugin! Thanks! |
@davama the best way to resurrect a pull request like this is to clone the repository yourself, checkout the pull request, resolve the merge conflicts, build it, put it into production in your environment, then share your results with others through a new pull request. That solves the problem for you and may allow others to benefit as well. |
(And in this case, basically recreate the patch without the massive reformatting.) |
Reverse-proxy-auth-plugin supports name & groups header injections but emails and display name are fetched from the LDAP server.
This fork add email and display name header injection support.
Thus we can manage authentication and user profile data with an httpd server.
What's more, using an httpd server as a reverse proxy, provides a simple solution to:
The original fork was made on tag 1.4.