diff options
author | Jan Cholasta <jcholast@redhat.com> | 2013-11-26 08:53:34 +0000 |
---|---|---|
committer | Petr Viktorin <pviktori@redhat.com> | 2013-11-26 12:58:17 +0100 |
commit | f20577ddc4ab40c2365c8abaa703d96019ec4eef (patch) | |
tree | f4bdd1e96da42d0e4cf66572443eb1326f9ecd4a /ipaserver/install | |
parent | 63d4f306867095654d1b46c8731a95140a5126ce (diff) | |
download | freeipa-f20577ddc4ab40c2365c8abaa703d96019ec4eef.tar.gz freeipa-f20577ddc4ab40c2365c8abaa703d96019ec4eef.tar.xz freeipa-f20577ddc4ab40c2365c8abaa703d96019ec4eef.zip |
Remove mod_ssl port workaround.
https://fedorahosted.org/freeipa/ticket/4021
Diffstat (limited to 'ipaserver/install')
-rw-r--r-- | ipaserver/install/httpinstance.py | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/ipaserver/install/httpinstance.py b/ipaserver/install/httpinstance.py index 689e657e2..e61a0c6d1 100644 --- a/ipaserver/install/httpinstance.py +++ b/ipaserver/install/httpinstance.py @@ -253,25 +253,24 @@ class HTTPInstance(service.Service): http_fd.close() os.chmod(target_fname, 0644) - def change_mod_nss_port_to_http(self): + def change_mod_nss_port_from_http(self): # mod_ssl enforces SSLEngine on for vhost on 443 even though # the listener is mod_nss. This then crashes the httpd as mod_nss # listened port obviously does not match mod_ssl requirements. # - # Change port to http to workaround the mod_ssl check, the SSL is - # enforced in the vhost later, so it is benign. + # The workaround for this was to change port to http. It is no longer + # necessary, as mod_nss now ships with default configuration which + # sets SSLEngine off when mod_ssl is installed. # - # Remove when https://bugzilla.redhat.com/show_bug.cgi?id=1023168 - # is fixed. - if not sysupgrade.get_upgrade_state('nss.conf', 'listen_port_updated'): - installutils.set_directive(NSS_CONF, 'Listen', '443 http', quotes=False) - sysupgrade.set_upgrade_state('nss.conf', 'listen_port_updated', True) + # Remove the workaround. + if sysupgrade.get_upgrade_state('nss.conf', 'listen_port_updated'): + installutils.set_directive(NSS_CONF, 'Listen', '443', quotes=False) + sysupgrade.set_upgrade_state('nss.conf', 'listen_port_updated', False) def __set_mod_nss_port(self): self.fstore.backup_file(NSS_CONF) if installutils.update_file(NSS_CONF, '8443', '443') != 0: print "Updating port in %s failed." % NSS_CONF - self.change_mod_nss_port_to_http() def __set_mod_nss_nickname(self, nickname): installutils.set_directive(NSS_CONF, 'NSSNickname', nickname) |