diff options
author | Endi S. Dewata <edewata@redhat.com> | 2017-07-07 17:23:54 +0200 |
---|---|---|
committer | Endi S. Dewata <edewata@redhat.com> | 2017-07-08 04:52:20 +0200 |
commit | eb08fba8a15e044802e9fa32ba0d4ed6c8eb31bc (patch) | |
tree | 189c8cc7f08d34301bba85c3e5e0b6eb0f582d27 /base/server/cms/src/com/netscape/cms/servlet | |
parent | 65b1242cd139e6306fb3e039193a3a6b223ea9b1 (diff) | |
download | pki-eb08fba8a15e044802e9fa32ba0d4ed6c8eb31bc.tar.gz pki-eb08fba8a15e044802e9fa32ba0d4ed6c8eb31bc.tar.xz pki-eb08fba8a15e044802e9fa32ba0d4ed6c8eb31bc.zip |
Refactored ConfigurationUtils.updateServerCertNickConf().
The code that updates serverCertNick.conf has been moved into
ConfigurationUtils.updateServerCertNickConf() and will only be
executed once.
https://pagure.io/dogtagpki/issue/2280
Change-Id: Iaae4429724ece683939aea8defe6fceeca237c4b
Diffstat (limited to 'base/server/cms/src/com/netscape/cms/servlet')
-rw-r--r-- | base/server/cms/src/com/netscape/cms/servlet/csadmin/ConfigurationUtils.java | 28 |
1 files changed, 17 insertions, 11 deletions
diff --git a/base/server/cms/src/com/netscape/cms/servlet/csadmin/ConfigurationUtils.java b/base/server/cms/src/com/netscape/cms/servlet/csadmin/ConfigurationUtils.java index 0c35c95cc..b8f771238 100644 --- a/base/server/cms/src/com/netscape/cms/servlet/csadmin/ConfigurationUtils.java +++ b/base/server/cms/src/com/netscape/cms/servlet/csadmin/ConfigurationUtils.java @@ -2849,17 +2849,6 @@ public class ConfigurationUtils { } } - String serverCertNickname = nickname; - String path = CMS.getConfigStore().getString("instanceRoot", ""); - if (certTag.equals("sslserver")) { - if (!CryptoUtil.isInternalToken(token)) { - serverCertNickname = token + ":" + nickname; - } - PrintStream ps = new PrintStream(path + "/conf/serverCertNick.conf", "UTF-8"); - ps.println(serverCertNickname); - ps.close(); - } - config.putString(subsystem + "." + certTag + ".nickname", nickname); config.putString(subsystem + "." + certTag + ".tokenname", token); if (certTag.equals("audit_signing")) { @@ -2884,6 +2873,23 @@ public class ConfigurationUtils { CMS.debug("updateConfig() done"); } + public static void updateServerCertNickConf() throws Exception { + + IConfigStore cs = CMS.getConfigStore(); + String token = cs.getString("preop.module.token"); + String nickname = getNickname(cs, "sslserver"); + + String serverCertNickname = nickname; + if (!CryptoUtil.isInternalToken(token)) { + serverCertNickname = token + ":" + nickname; + } + + String path = cs.getString("instanceRoot", ""); + PrintStream ps = new PrintStream(path + "/conf/serverCertNick.conf", "UTF-8"); + ps.println(serverCertNickname); + ps.close(); + } + public static String getNickname(IConfigStore config, String certTag) throws EBaseException { String instanceID = config.getString("instanceId", ""); |