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

server-install: remove broken no-pkinit check #690

Closed
wants to merge 1 commit into from

Conversation

stlaz
Copy link
Contributor

@stlaz stlaz commented Apr 4, 2017

Don't check for no-pkinit option in case pkinit cert file was
provided. Setting no-pkinit is prohibited in this case, so without
this fix we have an impossible option-check if we want to provide
an own pkinit certificate and private key.

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

Don't check for no-pkinit option in case pkinit cert file was
provided. Setting no-pkinit is prohibited in this case, so without
this fix we have an impossible option-check if we want to provide
an own pkinit certificate and private key.

https://pagure.io/freeipa/issue/6807
@martbab martbab self-assigned this Apr 7, 2017
@MartinBasti MartinBasti assigned MartinBasti and unassigned martbab Apr 10, 2017
@MartinBasti MartinBasti added the ack Pull Request approved, can be merged label Apr 12, 2017
@MartinBasti
Copy link
Contributor

master:

  • 1160dc5 server-install: remove broken no-pkinit check

ipa-4-5:

  • 1eb681e server-install: remove broken no-pkinit check

@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Apr 12, 2017
@stlaz stlaz deleted the install_nopkinit branch July 7, 2017 12:20
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
3 participants