diff options
| author | Petr Vobornik <pvoborni@redhat.com> | 2014-08-29 14:11:33 +0200 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2014-09-11 09:41:51 +0200 |
| commit | 97aebf8635ba26398353c91de9ca60844bea8c27 (patch) | |
| tree | 192fb591ad076ff7b74808cbd67873ec770451cb | |
| parent | de90d7d44930cfa615e50ed022de7ea48e31609a (diff) | |
| download | freeipa-97aebf8635ba26398353c91de9ca60844bea8c27.tar.gz freeipa-97aebf8635ba26398353c91de9ca60844bea8c27.tar.xz freeipa-97aebf8635ba26398353c91de9ca60844bea8c27.zip | |
install: create ff krb extension on every install, replica install and upgrade
We don't want to copy the extension from master to replica because the
replica may use newer version of FreeIPA and therefore the extension
code might be obsolete. Same reason for upgrades.
https://fedorahosted.org/freeipa/ticket/4478
Reviewed-By: Endi Sukma Dewata <edewata@redhat.com>
| -rwxr-xr-x | install/tools/ipa-replica-install | 5 | ||||
| -rw-r--r-- | ipaserver/install/httpinstance.py | 13 | ||||
| -rw-r--r-- | ipaserver/install/ipa_replica_prepare.py | 3 |
3 files changed, 2 insertions, 19 deletions
diff --git a/install/tools/ipa-replica-install b/install/tools/ipa-replica-install index a57b7c6a9..f7dfafc6b 100755 --- a/install/tools/ipa-replica-install +++ b/install/tools/ipa-replica-install @@ -244,11 +244,6 @@ def install_http(config, auto_redirect): if ipautil.file_exists(config.dir + "/configure.jar"): shutil.copy(config.dir + "/configure.jar", paths.CONFIGURE_JAR) - if ipautil.file_exists(config.dir + "/krb.js"): - shutil.copy(config.dir + "/krb.js", - paths.KRB_JS) - shutil.copy(config.dir + "/kerberosauth.xpi", - paths.KERBEROSAUTH_XPI) except Exception, e: print "error copying files: " + str(e) sys.exit(1) diff --git a/ipaserver/install/httpinstance.py b/ipaserver/install/httpinstance.py index 830ea486f..329dbb076 100644 --- a/ipaserver/install/httpinstance.py +++ b/ipaserver/install/httpinstance.py @@ -347,22 +347,13 @@ class HTTPInstance(service.Service): root_logger.warning('Object-signing certificate was not found; ' 'therefore, configure.jar was not created.') - self.setup_firefox_extension(self.realm, self.domain, force=True) + self.setup_firefox_extension(self.realm, self.domain) - def setup_firefox_extension(self, realm, domain, force=False): + def setup_firefox_extension(self, realm, domain): """Set up the signed browser configuration extension - - If the extension is already set up, skip the installation unless - ``force`` is true. """ target_fname = paths.KRB_JS - if os.path.exists(target_fname) and not force: - root_logger.info( - '%s exists, skipping install of Firefox extension', - target_fname) - return - sub_dict = dict(REALM=realm, DOMAIN=domain) db = certs.CertDB(realm) with open(db.passwd_fname) as pwdfile: diff --git a/ipaserver/install/ipa_replica_prepare.py b/ipaserver/install/ipa_replica_prepare.py index c1ced3dcb..7762614a1 100644 --- a/ipaserver/install/ipa_replica_prepare.py +++ b/ipaserver/install/ipa_replica_prepare.py @@ -370,9 +370,6 @@ class ReplicaPrepare(admintool.AdminTool): preferences_filename = paths.PREFERENCES_HTML if ipautil.file_exists(preferences_filename): self.copy_info_file(preferences_filename, "preferences.html") - self.copy_info_file(paths.KRB_JS, "krb.js") - self.copy_info_file( - paths.KERBEROSAUTH_XPI, "kerberosauth.xpi") jar_filename = paths.CONFIGURE_JAR if ipautil.file_exists(jar_filename): self.copy_info_file(jar_filename, "configure.jar") |