javascriptextjs
diff --git a/‎framework/config/src/org/apache/cloudstack/framework/config/dao/ConfigurationDaoImpl.java‎
Lines changed: 0 additions & 3 deletions b/‎framework/config/src/org/apache/cloudstack/framework/config/dao/ConfigurationDaoImpl.java‎
Lines changed: 0 additions & 3 deletions
diff --git a/‎framework/config/src/org/apache/cloudstack/framework/config/impl/ConfigurationVO.java‎
Lines changed: 15 additions & 3 deletions b/‎framework/config/src/org/apache/cloudstack/framework/config/impl/ConfigurationVO.java‎
Lines changed: 15 additions & 3 deletions
diff --git a/‎server/src/com/cloud/server/ConfigurationServerImpl.java‎
Lines changed: 1 addition & 2 deletions b/‎server/src/com/cloud/server/ConfigurationServerImpl.java‎
Lines changed: 1 addition & 2 deletions
@@ -210,9 +210,6 @@ public String getValueAndInitIfNotExist(String name, String category, String ini
                     update(name, category, initValue);
                 }
             } else {
-                if (category.equals("Hidden") || category.equals("Secure")) {
-                    initValue = DBEncryptionUtil.encrypt(initValue);
-                }
                 ConfigurationVO newConfig = new ConfigurationVO(category, "DEFAULT", "management-server", name, initValue, desc);
                 persist(newConfig);
             }
 
@@ -73,8 +73,8 @@ public ConfigurationVO(String category, String instance, String component, Strin
         this.instance = instance;
         this.component = component;
         this.name = name;
-        this.value = value;
         this.description = description;
+        setValue(value);
     }
 
     public ConfigurationVO(String component, ConfigKey<?> key) {
@@ -122,11 +122,23 @@ public void setName(String name) {
 
     @Override
     public String getValue() {
-        return (("Hidden".equals(getCategory()) || "Secure".equals(getCategory())) ? DBEncryptionUtil.decrypt(value) : value);
+        if(isEncryptedConfig()) {
+            return DBEncryptionUtil.decrypt(value);
+        } else {
+            return value;
+        }
     }
 
     public void setValue(String value) {
-        this.value = value;
+        if(isEncryptedConfig()) {
+            this.value = DBEncryptionUtil.encrypt(value);
+        } else {
+            this.value = value;
+        }
+    }
+
+    private boolean isEncryptedConfig() {
+        return "Hidden".equals(getCategory()) || "Secure".equals(getCategory());
     }
 
     @Override
 
@@ -203,7 +203,6 @@ public void persistDefaultValues() throws InternalErrorException {
                     String instance = "DEFAULT";
                     String component = c.getComponent();
                     String value = c.getDefaultValue();
-                    value = ("Hidden".equals(category) || "Secure".equals(category)) ? DBEncryptionUtil.encrypt(value) : value;
                     String description = c.getDescription();
                     ConfigurationVO configVO = new ConfigurationVO(category, instance, component, name, value, description);
                     configVO.setDefaultValue(value);
@@ -635,7 +634,7 @@ protected void updateSSLKeystore() {
                 }
                 String base64Keystore = getBase64Keystore(keystorePath);
                 ConfigurationVO configVO =
-                        new ConfigurationVO("Hidden", "DEFAULT", "management-server", "ssl.keystore", DBEncryptionUtil.encrypt(base64Keystore),
+                        new ConfigurationVO("Hidden", "DEFAULT", "management-server", "ssl.keystore", base64Keystore,
                                 "SSL Keystore for the management servers");
                 _configDao.persist(configVO);
                 s_logger.info("Stored SSL keystore to database.");
Original file line number	Diff line number	Diff line change
`@@ -210,9 +210,6 @@ public String getValueAndInitIfNotExist(String name, String category, String ini`
`210`	`210`	`update(name, category, initValue);`
`211`	`211`	`}`
`212`	`212`	`} else {`
`213`		`- if (category.equals("Hidden") \|\| category.equals("Secure")) {`
`214`		`- initValue = DBEncryptionUtil.encrypt(initValue);`
`215`		`- }`
`216`	`213`	`ConfigurationVO newConfig = new ConfigurationVO(category, "DEFAULT", "management-server", name, initValue, desc);`
`217`	`214`	`persist(newConfig);`
`218`	`215`	`}`