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

WebUI: cert login: Configure name of parameter used to pass username #704

Closed
wants to merge 1 commit into from
Closed

WebUI: cert login: Configure name of parameter used to pass username #704

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Apr 10, 2017

Directive LookupUserByCertificateParamName tells mod_lookup_identity module the
name of GET parameter that is used to provide username in case certificate is
mapped to multiple user accounts.
Without this directive login with certificate that's mapped to multiple users
doesn't work.

https://pagure.io/freeipa/issue/6860

Directive LookupUserByCertificateParamName tells mod_lookup_identity module the
name of GET parameter that is used to provide username in case certificate is
mapped to multiple user accounts.
Without this directive login with certificate that's mapped to multiple users
doesn't work.

https://pagure.io/freeipa/issue/6860
@flo-renaud flo-renaud self-assigned this Apr 10, 2017
@flo-renaud
Copy link
Contributor

Hi @dkupka
thank you for the patch, everything works as expected with a single certificate mapped to multiple users.

@flo-renaud flo-renaud added the ack Pull Request approved, can be merged label Apr 10, 2017
@pvomacka pvomacka added the pushed Pull Request has already been pushed label Apr 11, 2017
@pvomacka
Copy link

ipa-4-5:

  • a9721e5 WebUI: cert login: Configure name of parameter used to pass username
    master:

  • 157831a WebUI: cert login: Configure name of parameter used to pass username

@pvomacka pvomacka closed this Apr 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants