From 84b0cf08099b4a7b43cacb2ce89a160fa91deebd Mon Sep 17 00:00:00 2001
From: Daan Hoogland <dahn@onecht.net>
Date: Mon, 11 Jun 2018 09:19:26 +0000
Subject: [PATCH] comment on unencryption

---
 .../src/com/cloud/upgrade/dao/Upgrade41100to41110.java    | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/engine/schema/src/com/cloud/upgrade/dao/Upgrade41100to41110.java b/engine/schema/src/com/cloud/upgrade/dao/Upgrade41100to41110.java
index 50c01534c84..90e00016819 100644
--- a/engine/schema/src/com/cloud/upgrade/dao/Upgrade41100to41110.java
+++ b/engine/schema/src/com/cloud/upgrade/dao/Upgrade41100to41110.java
@@ -70,6 +70,14 @@ public class Upgrade41100to41110 implements DbUpgrade {
     }
 
     private void markUnnecessarySecureConfigsAsUnsecure(Connection conn) {
+        /*
+         * the following config items where added as 'Secure' in the past. For some this made sense but for the ones below,
+         * this makes no sense and is a inconvenience at best. The below method will
+         ** retrieve,
+         ** unencrypt,
+         ** mark as 'Advanced' and then
+         ** store the item
+         */
         String[] unsecureItems = new String[] {
                 "ldap.basedn",
                 "ldap.bind.principal",