From eb08fba8a15e044802e9fa32ba0d4ed6c8eb31bc Mon Sep 17 00:00:00 2001
From: "Endi S. Dewata" <edewata@redhat.com>
Date: Fri, 7 Jul 2017 17:23:54 +0200
Subject: 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
---
 .../cms/servlet/csadmin/ConfigurationUtils.java    | 28 +++++++++++++---------
 1 file changed, 17 insertions(+), 11 deletions(-)

(limited to 'base/server/cms/src/com')

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", "");
 
-- 
cgit