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

Add email and display name header injection support #30

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

efeat
Copy link

@efeat efeat commented Apr 4, 2017

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:

  • connect to several LDAP servers (even with different structures)
  • use technical users managed in the httpd server
  • create an SSO plateform with other integrated applications

The original fork was made on tag 1.4.

@MbaZZ
Copy link

MbaZZ commented Sep 18, 2017

Thank you, it's useful for we to add email directly on header.

@oleg-nenashev
Copy link
Member

@jenkinsci/code-reviewers

@ghost
Copy link

ghost commented Jun 8, 2019

Why hasn't this been picked up?
This would be really handy to be able to show my logged in users name properly via HTTP headers.

@davama
Copy link

davama commented May 18, 2023

Hello,

Thank you for the awesome plugin!
Noticed this pr and it's quite old...
Any chance for a resurrection?

Thanks!

@MarkEWaite
Copy link
Contributor

@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.

@jglick
Copy link
Member

jglick commented May 18, 2023

(And in this case, basically recreate the patch without the massive reformatting.)

sboardwell pushed a commit that referenced this pull request Jan 25, 2024
…30 (#124)

* Initial commit
* Apply suggestions from code review
* Added tests
* Fixed tests
* Pre commit changes
* fix: reformatting using mvn spotless:apply

---------
Co-authored-by: Steve Boardwell <steve.boardwell@gmail.com>
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.

7 participants