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

ipa-restore: remove /etc/httpd/conf.d/nss.conf #1672

Closed
wants to merge 1 commit into from

Conversation

flo-renaud
Copy link
Contributor

@flo-renaud flo-renaud commented Mar 12, 2018

When ipa-restore is called, it needs to delete the file
nss.conf, otherwise httpd server will try to initialize
the NSS engine and access NSSCertificateDatabase.
This is a regression introduced with the switch from NSS
to SSL.

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

@stlaz
Copy link
Contributor

stlaz commented Mar 13, 2018

The proper function of this PR depends on #1668

Copy link
Member

@tiran tiran left a comment

Choose a reason for hiding this comment

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

LGTM

Please land @stlaz PR first, then rebase.

When ipa-restore is called, it needs to delete the file
nss.conf, otherwise httpd server will try to initialize
the NSS engine and access NSSCertificateDatabase.
This is a regression introduced with the switch from NSS
to SSL.

https://pagure.io/freeipa/issue/7440
@flo-renaud
Copy link
Contributor Author

Rebased.

@tiran tiran added the ack Pull Request approved, can be merged label Mar 14, 2018
@tiran tiran added the pushed Pull Request has already been pushed label Mar 14, 2018
@tiran
Copy link
Member

tiran commented Mar 14, 2018

master:

  • 105e774 ipa-restore: remove /etc/httpd/conf.d/nss.conf

@tiran tiran closed this Mar 14, 2018
@flo-renaud flo-renaud deleted the t7440 branch May 15, 2018 15:42
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