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

httpinstance: clean up /etc/httpd/alias on uninstall #615

Closed
wants to merge 2 commits into from
Closed

httpinstance: clean up /etc/httpd/alias on uninstall #615

wants to merge 2 commits into from

Conversation

HonzaCholasta
Copy link
Contributor

@HonzaCholasta HonzaCholasta commented Mar 17, 2017

certs: do not implicitly create DS pin.txt

Do not implicitly create DS pin.txt in CertDB.init_from_pkcs12(), create
it explicitly in DSInstance.__enable_ssl().

This stops the file from being created in /etc/httpd/alias during classic
replica install.

httpinstance: clean up /etc/httpd/alias on uninstall

Restore cert8.db, key3.db, pwdfile.txt and secmod.db in /etc/httpd/alias
from backup on uninstall.

Files modified by IPA are kept with .ipasave suffix.

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

Jan Cholasta added 2 commits March 17, 2017 09:47
Do not implicitly create DS pin.txt in `CertDB.init_from_pkcs12()`, create
it explicitly in `DSInstance.__enable_ssl()`.

This stops the file from being created in /etc/httpd/alias during classic
replica install.

https://pagure.io/freeipa/issue/4639
Restore cert8.db, key3.db, pwdfile.txt and secmod.db in /etc/httpd/alias
from backup on uninstall.

Files modified by IPA are kept with .ipasave suffix.

https://pagure.io/freeipa/issue/4639
@martbab martbab self-assigned this Mar 20, 2017
@martbab martbab added the ack Pull Request approved, can be merged label Mar 22, 2017
@martbab
Copy link
Contributor

martbab commented Mar 22, 2017

ipa-4-5:

  • cf188c8 certs: do not implicitly create DS pin.txt

  • f788e3e httpinstance: clean up /etc/httpd/alias on uninstall
    master:

  • bbd18cf certs: do not implicitly create DS pin.txt

  • e263cb4 httpinstance: clean up /etc/httpd/alias on uninstall

@martbab martbab added the pushed Pull Request has already been pushed label Mar 22, 2017
@martbab martbab closed this Mar 22, 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