From 392cf3d4e75debfb8ebb55525404ff71793b7d5b Mon Sep 17 00:00:00 2001
From: luukas <lukasdoesdev@gmail.com>
Date: Tue, 14 Jun 2022 01:35:43 +0300
Subject: [PATCH 01/13] Make the file editor stick to the top of the page

---
 .../templates/panel/server_files.html         | 53 +++++++++++--------
 1 file changed, 32 insertions(+), 21 deletions(-)

diff --git a/app/frontend/templates/panel/server_files.html b/app/frontend/templates/panel/server_files.html
index 66ed9c86..4603fd24 100644
--- a/app/frontend/templates/panel/server_files.html
+++ b/app/frontend/templates/panel/server_files.html
@@ -211,29 +211,40 @@
               .tree-nested {
                 display: none;
               }
+
+              html, body, body > .container-scroller {
+                overflow: initial;
+              }
+
+              .editorManager {
+                top: 63px;
+                position: sticky;
+              }
             </style>
             <div class="col-md-6 col-sm-12">
-              <h2 id="fileError"></h2>
-              <div id="editorParent">
-                {{ translate('serverFiles', 'editingFile', data['lang']) }} <span id="editingFile"></span>
-                <div id="editor" onresize="editor.resize()" style="resize: both;width: 100%;">file_contents</div>
-                <br />
+              <div class="editorManager">
+                <h2 id="fileError"></h2>
+                <div id="editorParent">
+                  {{ translate('serverFiles', 'editingFile', data['lang']) }} <span id="editingFile"></span>
+                  <div id="editor" onresize="editor.resize()" style="resize: both;width: 100%;">file_contents</div>
+                  <br />
+                </div>
+                {{ translate('serverFiles', 'keybindings', data['lang']) }}:
+                <div class="btn-group" role="group">
+                  <button onclick="setKeyboard(event.target)" class="btn btn-primary" data-handler-name="null">{{
+                    translate('serverFiles', 'default', data['lang']) }}</button>
+                  <button onclick="setKeyboard(event.target)" class="btn btn-secondary"
+                    data-handler-name="ace/keyboard/vim">Vim</button>
+                  <button onclick="setKeyboard(event.target)" class="btn btn-secondary"
+                    data-handler-name="ace/keyboard/emacs">Emacs</button>
+                  <button onclick="setKeyboard(event.target)" class="btn btn-secondary"
+                    data-handler-name="ace/keyboard/sublime">Sublime</button>
+                </div>
+                <h3 id="file_warn"></h3>
+                <button class="btn btn-success" onclick="save()"><i class="fas fa-save"></i> {{ translate('serverFiles',
+                  'save', data['lang']) }}</button>
+                <span style="color: #2fb689; margin-left: 10px;" id="save_status"></span>
               </div>
-              {{ translate('serverFiles', 'keybindings', data['lang']) }}:
-              <div class="btn-group" role="group">
-                <button onclick="setKeyboard(event.target)" class="btn btn-primary" data-handler-name="null">{{
-                  translate('serverFiles', 'default', data['lang']) }}</button>
-                <button onclick="setKeyboard(event.target)" class="btn btn-secondary"
-                  data-handler-name="ace/keyboard/vim">Vim</button>
-                <button onclick="setKeyboard(event.target)" class="btn btn-secondary"
-                  data-handler-name="ace/keyboard/emacs">Emacs</button>
-                <button onclick="setKeyboard(event.target)" class="btn btn-secondary"
-                  data-handler-name="ace/keyboard/sublime">Sublime</button>
-              </div>
-              <h3 id="file_warn"></h3>
-              <button class="btn btn-success" onclick="save()"><i class="fas fa-save"></i> {{ translate('serverFiles',
-                'save', data['lang']) }}</button>
-              <span style="color: #2fb689; margin-left: 10px;" id="save_status"></span>
             </div>
           </div>
 
@@ -1113,4 +1124,4 @@
 
 </script>
 
-{% end %}
\ No newline at end of file
+{% end %}

From a67f069709417dd213911af1148b995a64ee215e Mon Sep 17 00:00:00 2001
From: luukas <lukasdoesdev@gmail.com>
Date: Tue, 14 Jun 2022 02:45:17 +0300
Subject: [PATCH 02/13] Fix Ace editor saving keybinds and ex-commands

* Remove our manual keydown handler and use an Ace builtin function
* Define Vim's :write aka :w ex-command
---
 .../templates/panel/server_files.html         | 28 ++++++++++++-------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/app/frontend/templates/panel/server_files.html b/app/frontend/templates/panel/server_files.html
index 4603fd24..d6cdf220 100644
--- a/app/frontend/templates/panel/server_files.html
+++ b/app/frontend/templates/panel/server_files.html
@@ -278,6 +278,17 @@
   let editor = ace.edit('editor');
   editor.setTheme('ace/theme/dracula');
   editor.session.setUseSoftTabs(true);
+  editor.commands.addCommand({
+    name: 'saveFile',
+    bindKey: {
+      win: 'Ctrl-S',
+      mac: 'Command-S',
+      sender: 'editor|cli'
+    },
+    exec: function(env, args, request) {
+      save()
+    }
+  });
 
   // mouseup = css resize end
   document.addEventListener("mouseup", function (e) {
@@ -534,15 +545,6 @@
 
   });
 
-  $('#editor').bind('keydown', function (event) {
-    file_loaded = true;
-    if (event.ctrlKey && event.key === 's') {
-      event.preventDefault();
-      save();
-    }
-    //console.log(event.keyCode);
-  });
-
 
   setFileName();
   $('#editorParent').toggle(false) // show
@@ -1108,7 +1110,13 @@
   function setKeyboard(target) {
     var handlerName = target.getAttribute('data-handler-name');
     if (handlerName == 'null') handlerName = null;
-    editor.setKeyboardHandler(handlerName);
+    editor.setKeyboardHandler(handlerName, () => {
+      if (handlerName == 'ace/keyboard/vim') {
+        require("ace/keyboard/vim").Vim.defineEx('write', 'w', function() {
+          save();
+        });
+      }
+    });
 
     var nodes = target.parentNode.querySelectorAll("[data-handler-name]");
     nodes.forEach(node => {

From c9744554d47da4199e4844fe29908321a667a83c Mon Sep 17 00:00:00 2001
From: luukas <lukasdoesdev@gmail.com>
Date: Tue, 14 Jun 2022 03:07:26 +0300
Subject: [PATCH 03/13] Improve the code for save status

---
 .../templates/panel/server_files.html         | 43 ++++++-------------
 1 file changed, 14 insertions(+), 29 deletions(-)

diff --git a/app/frontend/templates/panel/server_files.html b/app/frontend/templates/panel/server_files.html
index d6cdf220..45e02ced 100644
--- a/app/frontend/templates/panel/server_files.html
+++ b/app/frontend/templates/panel/server_files.html
@@ -386,8 +386,7 @@
     },
   ];
 
-  var filePath = '';
-  let file_loaded = false;
+  let filePath = '', serverFileContent = '';
 
   function clickOnFile(event) {
     filePath = event.target.getAttribute('data-path');
@@ -408,7 +407,7 @@
           $('#fileError').toggle(false)   // hide
           setFileName(event.target.innerText);
           editor.session.setValue(json.content);
-          file_loaded = false;
+          serverFileContent = json.content;
         }
       },
     });
@@ -523,27 +522,14 @@
       timer = null;
     };
   }
-  editor.on('change', function (event) {
-    if (!event.ctrlKey && !event.shiftKey) {
-      if (file_loaded) {
-        document.getElementById('save_status').innerHTML = '<i class="fal fa-file"></i>';
-        document.getElementById('save_status').style.color = 'gray';
-      } else {
-        document.getElementById('save_status').innerHTML = '<i class="fal fa-file-check"></i>';
-        document.getElementById('save_status').style.color = '#2fb689';
-      }
-    }
-  });
-  editor.on('undo', function (event) {
-    document.getElementById('save_status').innerHTML = '<i class="fal fa-file-xmark"></i>';
-    document.getElementById('save_status').style.color = 'gray';
-
-  });
-  editor.on('redo', function (event) {
-    document.getElementById('save_status').innerHTML = '<i class="fal fa-file-xmark"></i>';
-    document.getElementById('save_status').style.color = 'gray';
-
-  });
+  /**
+    * @param {boolean} saved
+  */
+  const setSaveStatus = (saved) => {
+    document.getElementById('save_status').innerHTML = `<i class="fal ${saved ? "fa-file-check" : "fa-file"}"></i>`;
+    document.getElementById('save_status').style.color = saved ? '#2fb689' : 'gray';
+  }
+  ['change', 'undo', 'redo'].forEach(event => editor.on(event, (event) => setSaveStatus(serverFileContent === editor.session.getValue())))
 
 
   setFileName();
@@ -589,11 +575,10 @@
         file_contents: text,
         file_path: filePath
       },
-      success: function (data) {
-        console.log("got response:");
-        document.getElementById("save_status").innerHTML = '<i class="fal fa-file-check"></i>';
-        document.getElementById('save_status').style.color = '#2fb689';
-      },
+      success: (data) => {
+        serverFileContent = text;
+        setSaveStatus(true)
+      }
     });
   }
 

From cd8ebfdcbe61ab461ef591ed98bba2c39614f12b Mon Sep 17 00:00:00 2001
From: luukas <lukasdoesdev@gmail.com>
Date: Thu, 16 Jun 2022 00:10:48 +0300
Subject: [PATCH 04/13] Make the WebSocket automatically reconnect.

The WebSocket reconnects 5 seconds after the connection's closure.
If the connection reconnection fails it will try to reconnect after
another 5 seconds.
---
 app/frontend/templates/base.html | 133 ++++++++++++++++++-------------
 1 file changed, 78 insertions(+), 55 deletions(-)

diff --git a/app/frontend/templates/base.html b/app/frontend/templates/base.html
index 4b28120a..6d8c7332 100755
--- a/app/frontend/templates/base.html
+++ b/app/frontend/templates/base.html
@@ -164,7 +164,7 @@
 
   <script>
     $.extend($.fn.dataTable.defaults, {
-      language: {% raw translate('datatables', 'i18n', data['lang']) %}
+      // {{ '\nlanguage:' }} {% raw translate('datatables', 'i18n', data['lang']) %}
       })
 
     //used to get cookies from browser - this is part of tornados xsrf protection - it's for extra security
@@ -192,60 +192,90 @@
       });
     });
 
+    let usingWebSockets = false;
+    let webSocket = null;
     // {% if request.protocol == 'https' %}
-    let usingWebSockets = true;
+    usingWebSockets = true;
 
     let listenEvents = [];
+    let wsOpen = false;
+    /**
+      * @type {number | null} reconnectorId An interval ID for the reconnector.
+      */
+    let reconnectorId = null;
 
-    try {
-      pageQueryParams = 'page_query_params=' + encodeURIComponent(location.search)
-      page = 'page=' + encodeURIComponent(location.pathname)
-      var wsInternal = new WebSocket('wss://' + location.host + '/ws?' + page + '&' + pageQueryParams);
-      wsInternal.onopen = function () {
-        console.log('opened WebSocket connection:', wsInternal)
-      };
-      wsInternal.onmessage = function (rawMessage) {
-        var message = JSON.parse(rawMessage.data);
+    const wsPageQueryParams = 'page_query_params=' + encodeURIComponent(location.search)
+    const wsPage = 'page=' + encodeURIComponent(location.pathname)
 
-        console.log('got message: ', message)
+    const sendWssError = () => wsOpen || warn(
+      'WebSockets are required for Crafty to work. This websocket connection has been closed. Are you using a reverse proxy?',
+      'https://wiki.craftycontrol.com/en/4/docs/Reverse%20Proxy%20Examples',
+      'wssError'
+    )
 
-        listenEvents
-          .filter(listenedEvent => listenedEvent.event == message.event)
-          .forEach(listenedEvent => listenedEvent.callback(message.data))
-      };
-      wsInternal.onerror = function (errorEvent) {
-        console.error('WebSocket Error', errorEvent);
-        warn('WebSockets are required for Crafty to work. This websocket connection has been closed. Are you using a reverse proxy?', 'https://wiki.craftycontrol.com/en/4/docs/Reverse%20Proxy%20Examples')
-
-      };
-      wsInternal.onclose = function (closeEvent) {
-        console.log('Closed WebSocket', closeEvent);
-        setTimeout(sendWssError, 7000);
-      };
-
-
-      webSocket = {
-        on: function (event, callback) {
-          console.log('registered ' + event + ' event');
-          listenEvents.push({ event: event, callback: callback })
-        },
-        emit: function (event, data) {
-          var message = {
-            event: event,
-            data: data
+    function startWebSocket() {
+      console.log('%c[Crafty Controller] %cConnecting the WebSocket', 'font-weight: 900; color: #800080;', 'font-weight: 900; color: #eee;');
+      try {
+        var wsInternal = new WebSocket('wss://' + location.host + '/ws?' + wsPage + '&' + wsPageQueryParams);
+        wsInternal.onopen = function () {
+          console.log('opened WebSocket connection:', wsInternal)
+          wsOpen = true;
+          if (typeof reconnectorId === 'number') {
+            document.querySelectorAll('.wssError').forEach(el => el.remove())
+            clearInterval(reconnectorId);
+            reconnectorId = null;
           }
+        };
+        wsInternal.onmessage = function (rawMessage) {
+          var message = JSON.parse(rawMessage.data);
 
-          wsInternal.send(JSON.stringify(message));
+          console.log('got message: ', message)
+
+          listenEvents
+            .filter(listenedEvent => listenedEvent.event == message.event)
+            .forEach(listenedEvent => listenedEvent.callback(message.data))
+        };
+        wsInternal.onerror = function (errorEvent) {
+          console.error('WebSocket Error', errorEvent);
+        };
+        wsInternal.onclose = function (closeEvent) {
+          wsOpen = false;
+          console.log('Closed WebSocket', closeEvent);
+
+          if (typeof reconnectorId !== 'number') {
+            // Discard old websocket and create a new one in 5 seconds
+            wsInternal = null
+            reconnectorId = setInterval(startWebSocket, 5000)
+            setTimeout(sendWssError, 7000);
+          } else {
+            wsInternal.close();
+          }
+        };
+
+
+        webSocket = {
+          on: function (event, callback) {
+            console.log('registered ' + event + ' event');
+            listenEvents.push({ event: event, callback: callback })
+          },
+          emit: function (event, data) {
+            var message = {
+              event: event,
+              data: data
+            }
+
+            wsInternal.send(JSON.stringify(message));
+          }
         }
+      } catch (error) {
+        console.error('Error while making websocket helpers', error);
+        usingWebSockets = false;
       }
-    } catch (error) {
-      console.error('Error while making websocket helpers', error);
-      usingWebSockets = false;
     }
+
+    startWebSocket();
     // {% else %}
-    let usingWebSockets = false;
     warn('WebSockets are not supported in Crafty if not using the https protocol')
-    var webSocket;
     // {% end%}
 
     if (webSocket) {
@@ -265,8 +295,6 @@
           }
         })
       });
-    }
-    if (webSocket) {
       webSocket.on('support_status_update', function (logs) {
         if (logs.percent >= 100) {
           document.getElementById('logs_progress_bar').innerHTML = '100%';
@@ -276,8 +304,6 @@
           document.getElementById('logs_progress_bar').style.width = logs.percent + '%';
         }
       });
-    }
-    if (webSocket) {
       webSocket.on('send_logs_bootbox', function (server_id) {
         var x = document.querySelector('.bootbox');
         if (x) {
@@ -302,9 +328,6 @@
           }
         });
       });
-    }
-
-    if (webSocket) {
       webSocket.on('send_eula_bootbox', function (server_id) {
         var x = document.querySelector('.bootbox');
         if (x) {
@@ -339,10 +362,6 @@
       });
     }
 
-    function sendWssError(){
-        warn('WebSockets are required for Crafty to work. This websocket connection has been closed. Are you using a reverse proxy?', 'https://wiki.craftycontrol.com/en/4/docs/Reverse%20Proxy%20Examples')
-    }
-
     function eulaAgree(server_id, command) {
       //< !--this getCookie function is in base.html-- >
       var token = getCookie("_xsrf");
@@ -360,7 +379,7 @@
     }
 
 
-    function warn(message, link = null) {
+    function warn(message, link = null, className = null) {
       var closeEl = document.createElement('span');
       var strongEL = document.createElement('strong');
       var msgEl = document.createElement('div');
@@ -397,6 +416,10 @@
         parentEl.appendChild(linkEl);
       }
 
+      if (className) {
+        parentEl.classList.add(className);
+      }
+
       document.querySelector('.warnings').appendChild(parentEl);
     }
 
@@ -473,4 +496,4 @@
 
 </body>
 
-</html>
\ No newline at end of file
+</html>

From d9c03caa9b2cfa17aae655eac21b7ac23eab3597 Mon Sep 17 00:00:00 2001
From: luukas <lukasdoesdev@gmail.com>
Date: Thu, 16 Jun 2022 00:19:48 +0300
Subject: [PATCH 05/13] Fix an error when there are no servers

---
 app/frontend/templates/panel/dashboard.html | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/frontend/templates/panel/dashboard.html b/app/frontend/templates/panel/dashboard.html
index 34cddf63..8c215a77 100644
--- a/app/frontend/templates/panel/dashboard.html
+++ b/app/frontend/templates/panel/dashboard.html
@@ -835,7 +835,8 @@
       "ordering": false, // false to disable sorting (or any other option)
       "paging": false
     });
-    document.getElementById('first').setAttribute('draggable', false);
+    const first = document.querySelector('#first')
+    first !== null && first.setAttribute('draggable', false);
     $('.dataTables_length').addClass('bs-select');
   });
   /* Search Bar End */
@@ -908,4 +909,4 @@
 
 </script>
 
-{% end %}
\ No newline at end of file
+{% end %}

From f39e75616ce1963906daf8df443e5096ead30995 Mon Sep 17 00:00:00 2001
From: luukas <lukasdoesdev@gmail.com>
Date: Thu, 16 Jun 2022 02:37:53 +0300
Subject: [PATCH 06/13] Use an exponential backoff algorhithm.

https://stackoverflow.com/a/37038217/15388424
---
 app/frontend/templates/base.html | 34 +++++++++++++++++++++++++++-----
 1 file changed, 29 insertions(+), 5 deletions(-)

diff --git a/app/frontend/templates/base.html b/app/frontend/templates/base.html
index 6d8c7332..eba6c2e8 100755
--- a/app/frontend/templates/base.html
+++ b/app/frontend/templates/base.html
@@ -163,6 +163,27 @@
   <script type="text/javascript" src="/static/assets/js/motd.js"></script>
 
   <script>
+    /**
+      * Returns a random number between min (inclusive) and max (exclusive)
+      */
+    function getRandomArbitrary(min, max) {
+      return Math.random() * (max - min) + min;
+    }
+
+    /**
+      * Returns a random integer between min (inclusive) and max (inclusive).
+      * The value is no lower than min (or the next integer greater than min
+      * if min isn't an integer) and no greater than max (or the next integer
+      * lower than max if max isn't an integer).
+      * Using Math.round() will give you a non-uniform distribution!
+      */
+    function getRandomInt(min, max) {
+      min = Math.ceil(min);
+      max = Math.floor(max);
+      return Math.floor(Math.random() * (max - min + 1)) + min;
+    }
+
+
     $.extend($.fn.dataTable.defaults, {
       // {{ '\nlanguage:' }} {% raw translate('datatables', 'i18n', data['lang']) %}
       })
@@ -203,6 +224,7 @@
       * @type {number | null} reconnectorId An interval ID for the reconnector.
       */
     let reconnectorId = null;
+    let failedConnectionCounter = 0; // https://stackoverflow.com/a/37038217/15388424
 
     const wsPageQueryParams = 'page_query_params=' + encodeURIComponent(location.search)
     const wsPage = 'page=' + encodeURIComponent(location.pathname)
@@ -220,6 +242,7 @@
         wsInternal.onopen = function () {
           console.log('opened WebSocket connection:', wsInternal)
           wsOpen = true;
+          failedConnectionCounter = 0;
           if (typeof reconnectorId === 'number') {
             document.querySelectorAll('.wssError').forEach(el => el.remove())
             clearInterval(reconnectorId);
@@ -243,13 +266,14 @@
           console.log('Closed WebSocket', closeEvent);
 
           if (typeof reconnectorId !== 'number') {
-            // Discard old websocket and create a new one in 5 seconds
-            wsInternal = null
-            reconnectorId = setInterval(startWebSocket, 5000)
             setTimeout(sendWssError, 7000);
-          } else {
-            wsInternal.close();
           }
+          console.info("Reconnecting with a timeout of", (getRandomArbitrary(0, 2 ** failedConnectionCounter - 1) + 5) * 1000, "milliseconds");
+          // Discard old websocket and create a new one in 5 seconds
+          wsInternal = null
+          reconnectorId = setTimeout(startWebSocket, (getRandomArbitrary(0, 2 ** failedConnectionCounter - 1) + 5) * 1000)
+
+          failedConnectionCounter++;
         };
 
 

From 581b6734d399a4826cbc5bc6465a067ad557af88 Mon Sep 17 00:00:00 2001
From: luukas <lukasdoesdev@gmail.com>
Date: Thu, 16 Jun 2022 15:31:11 +0300
Subject: [PATCH 07/13] Remove unused server_uuid variables to fix Pylint

---
 app/classes/web/routes/api/servers/server/action.py | 1 -
 app/classes/web/server_handler.py                   | 1 -
 2 files changed, 2 deletions(-)

diff --git a/app/classes/web/routes/api/servers/server/action.py b/app/classes/web/routes/api/servers/server/action.py
index 2360b172..cf9163b9 100644
--- a/app/classes/web/routes/api/servers/server/action.py
+++ b/app/classes/web/routes/api/servers/server/action.py
@@ -46,7 +46,6 @@ class ApiServersServerActionHandler(BaseApiHandler):
             return Servers.select().where(Servers.server_name == name).exists()
 
         server_data = self.controller.servers.get_server_data_by_id(server_id)
-        server_uuid = server_data.get("server_uuid")
         new_server_name = server_data.get("server_name") + " (Copy)"
 
         name_counter = 1
diff --git a/app/classes/web/server_handler.py b/app/classes/web/server_handler.py
index e2913f5f..bf4e930d 100644
--- a/app/classes/web/server_handler.py
+++ b/app/classes/web/server_handler.py
@@ -201,7 +201,6 @@ class ServerHandler(BaseHandler):
                     server_data = self.controller.servers.get_server_data_by_id(
                         server_id
                     )
-                    server_uuid = server_data.get("server_uuid")
                     new_server_name = server_data.get("server_name") + " (Copy)"
 
                     name_counter = 1

From 35518e960ff94badde9b7a17441c8036eb6bc0a1 Mon Sep 17 00:00:00 2001
From: amcmanu3 <mcdeweykp@gmail.com>
Date: Thu, 16 Jun 2022 09:52:21 -0400
Subject: [PATCH 08/13] Fix username capital issues once and for all

---
 app/classes/web/api_handler.py            | 2 +-
 app/classes/web/panel_handler.py          | 4 ++--
 app/classes/web/routes/api/users/index.py | 1 +
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/app/classes/web/api_handler.py b/app/classes/web/api_handler.py
index ae525dbb..43af4ae8 100644
--- a/app/classes/web/api_handler.py
+++ b/app/classes/web/api_handler.py
@@ -338,7 +338,7 @@ class CreateUser(ApiHandler):
             self.access_denied(user)
             return
 
-        new_username = self.get_argument("username")
+        new_username = self.get_argument("username").lower()
         new_pass = self.get_argument("password")
 
         if new_username:
diff --git a/app/classes/web/panel_handler.py b/app/classes/web/panel_handler.py
index ed8f3a73..94cd12a7 100644
--- a/app/classes/web/panel_handler.py
+++ b/app/classes/web/panel_handler.py
@@ -1774,7 +1774,7 @@ class PanelHandler(BaseHandler):
                     "system user is not editable"
                 )
             user_id = bleach.clean(self.get_argument("id", None))
-            username = bleach.clean(self.get_argument("username", None))
+            username = bleach.clean(self.get_argument("username", None).lower())
             password0 = bleach.clean(self.get_argument("password0", None))
             password1 = bleach.clean(self.get_argument("password1", None))
             email = bleach.clean(self.get_argument("email", "default@example.com"))
@@ -1943,7 +1943,7 @@ class PanelHandler(BaseHandler):
             self.finish()
 
         elif page == "add_user":
-            username = bleach.clean(self.get_argument("username", None))
+            username = bleach.clean(self.get_argument("username", None).lower())
             if username.lower() == "system":
                 self.redirect(
                     "/panel/error?error=Unauthorized access: "
diff --git a/app/classes/web/routes/api/users/index.py b/app/classes/web/routes/api/users/index.py
index 4c5a85a2..3e4cfdab 100644
--- a/app/classes/web/routes/api/users/index.py
+++ b/app/classes/web/routes/api/users/index.py
@@ -95,6 +95,7 @@ class ApiUsersIndexHandler(BaseApiHandler):
             )
 
         username = data["username"]
+        username = str(username).lower()
         password = data["password"]
         email = data.get("email", "default@example.com")
         enabled = data.get("enabled", True)

From c0006e47f4752a65aa5714e04d45b0aedae9d1cc Mon Sep 17 00:00:00 2001
From: Zedifus <zedifus@zediverse.gg>
Date: Thu, 16 Jun 2022 16:45:08 +0100
Subject: [PATCH 09/13] Add version inheretence & config check
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Also add pretty colours

Also config check will always return true but meh 🤷‍♂️
If api key location changes it will be accurate
---
 docker_launcher.sh | 27 ++++++++++++++++++++-------
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/docker_launcher.sh b/docker_launcher.sh
index ec842f4c..3e2cfd6c 100644
--- a/docker_launcher.sh
+++ b/docker_launcher.sh
@@ -2,7 +2,7 @@
 
 # Check if config exists taking one from image if needed.
 if [ ! "$(ls -A --ignore=.gitkeep ./app/config)" ]; then
-    echo "Wrapper | 🏗️ Config not found, pulling defaults..."
+    echo "\033[36mWrapper | \033[33m🏗️  Config not found, pulling defaults..."
     mkdir ./app/config/ 2> /dev/null
     cp -r ./app/config_original/* ./app/config/
 
@@ -10,11 +10,24 @@ if [ ! "$(ls -A --ignore=.gitkeep ./app/config)" ]; then
         # We're running as root;
         # Look for files & dirs that require group permissions to be fixed
         # This will do the full /crafty dir, so will take a miniute.
-        echo "Wrapper | 📋 Looking for problem bind mount permissions globally..."
+        echo "\033[36mWrapper | \033[35m📋 Looking for problem bind mount permissions globally..."
         find . ! -group root -exec chgrp root {} \;
         find . ! -perm g+rw -exec chmod g+rw {} \;
         find . -type d ! -perm g+s -exec chmod g+s {} \;
     fi
+else
+    # Keep version file up to date with image
+    cp -f ./app/config_original/version.json ./app/config/version.json
+
+    # Compare if user's config is different from image, and show differences.
+    echo "\033[36mWrapper | \033[35m🏗️  Checking for config.json changes..."
+    cp -f ./app/config_original/config.json ./app/config/config_image_template
+    if [ "$(diff -q ./app/config/config.json ./app/config/config_image_template)" ]; then
+        echo "\033[36mWrapper | \033[33m👷 We've found differences in your local config, please review!,"
+        echo "\033[36m        | \033[33m   (This could be an outdated config.json)"
+    else
+        echo "\033[36mWrapper | \033[32m✅ Config good! Proceeding..."
+    fi
 fi
 
 
@@ -24,21 +37,21 @@ if [ $(id -u) -eq 0 ]; then
     # If we find files in import directory, we need to ensure all dirs are owned by the root group,
     # This fixes bind mounts that may have incorrect perms.
     if [ "$(ls -A --ignore=.gitkeep ./import)" ]; then
-        echo "Wrapper | 📋 Files present in import, checking/fixing permissions..."
-        echo "Wrapper | ⏳ Please be paitent for larger servers..."
+        echo "\033[36mWrapper | \033[35m📋 Files present in import directory, checking/fixing permissions..."
+        echo "\033[36mWrapper | \033[33m⏳ Please be paitent for larger servers..."
         find . ! -group root -exec chgrp root {} \;
         find . ! -perm g+rw -exec chmod g+rw {} \;
         find . -type d ! -perm g+s -exec chmod g+s {} \;
-        echo "Wrapper | ✅ Permissions Fixed! (This will happen every boot until /import is empty!)"
+        echo "\033[36mWrapper | \033[32m✅ Permissions Fixed! (This will happen every boot until /import is empty!)"
     fi
 
     # Switch user, activate our prepared venv and lauch crafty
     args="$@"
-    echo "Wrapper | 🚀 Launching crafty with [$args]"
+    echo "\033[36mWrapper | \033[32m🚀 Launching crafty with [\033[34m$args\033[32m]"
     exec sudo -u crafty bash -c "source ./.venv/bin/activate && exec python3 main.py $args"
 else
     # Activate our prepared venv
-    echo "Wrapper | 🚀 Non-root host detected, using normal exec"
+    echo "\033[36mWrapper | \033[32m🚀 Non-root host detected, using normal exec"
     . ./.venv/bin/activate
     # Use exec as our perms are already correct
     # This is likely if using Kubernetes/OpenShift etc

From e3d289e6fdb6672f45333c92ad0b376d9ea86e76 Mon Sep 17 00:00:00 2001
From: amcmanu3 <mcdeweykp@gmail.com>
Date: Thu, 16 Jun 2022 12:11:21 -0400
Subject: [PATCH 10/13] Fix support logs download issues

---
 app/classes/shared/main_controller.py | 15 +++++++++++++--
 app/classes/web/panel_handler.py      |  2 --
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/app/classes/shared/main_controller.py b/app/classes/shared/main_controller.py
index f1515031..39db11cd 100644
--- a/app/classes/shared/main_controller.py
+++ b/app/classes/shared/main_controller.py
@@ -82,8 +82,16 @@ class Controller:
         if exec_user["preparing"]:
             return
         self.users.set_prepare(exec_user["user_id"])
-        # Delete previous instace of logs
-        self.del_support_file(exec_user["support_logs"])
+        logger.info("Checking for previous support logs.")
+        if exec_user["support_logs"] != "":
+            logger.info(
+                f"Found previous support log request at {exec_user['support_logs']}"
+            )
+            if self.helper.validate_traversal(
+                tempfile.gettempdir(), exec_user["support_logs"]
+            ):
+                logger.debug("No transversal detected. Going for the delete.")
+                self.del_support_file(exec_user["support_logs"])
         # pausing so on screen notifications can run for user
         time.sleep(7)
         self.helper.websocket_helper.broadcast_user(
@@ -177,6 +185,9 @@ class Controller:
     def del_support_file(self, temp_zip_storage):
         try:
             FileHelpers.del_file(temp_zip_storage)
+            logger.info(
+                f"Old support logs successfully deleted from {temp_zip_storage}"
+            )
         except FileNotFoundError:
             logger.info("No temp file found. Assuming it's already been cleaned up")
         except PermissionError:
diff --git a/app/classes/web/panel_handler.py b/app/classes/web/panel_handler.py
index ed8f3a73..cf28316a 100644
--- a/app/classes/web/panel_handler.py
+++ b/app/classes/web/panel_handler.py
@@ -1223,8 +1223,6 @@ class PanelHandler(BaseHandler):
 
         elif page == "download_support_package":
             temp_zip_storage = exec_user["support_logs"]
-            # We'll reset the support path for this user now.
-            self.controller.users.set_support_path(exec_user["user_id"], "")
 
             self.set_header("Content-Type", "application/octet-stream")
             self.set_header(

From 91de96194cf2b4efbe6061069a3daf362b40f35f Mon Sep 17 00:00:00 2001
From: Zedifus <zedifus@zediverse.gg>
Date: Thu, 16 Jun 2022 18:33:57 +0100
Subject: [PATCH 11/13] Add release pipes & CHANGELOG.md

Also add checkbox for changelog updates in MR
---
 .gitlab-ci.yml                             |  2 ++
 .gitlab/merge_request_templates/Default.md |  1 +
 .gitlab/release.yml                        | 16 +++++++++++
 .gitlab/windows-build.yml                  |  4 +++
 CHANGELOG.md                               | 31 ++++++++++++++++++++++
 5 files changed, 54 insertions(+)
 create mode 100644 .gitlab/release.yml
 create mode 100644 CHANGELOG.md

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 4ec9ed84..03144b48 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -7,6 +7,7 @@ stages:
   - test
   - prod-deployment
   - dev-deployment
+  - release
 
 variables:
   DOCKER_HOST: tcp://docker:2376
@@ -16,6 +17,7 @@ include:
   - local: .gitlab/lint.yml
   - local: .gitlab/docker-build.yml
   - local: .gitlab/windows-build.yml
+  - local: .gitlab/release.yml
   - template: Security/Dependency-Scanning.gitlab-ci.yml
   - template: Security/SAST.gitlab-ci.yml
   - template: Security/Secret-Detection.gitlab-ci.yml
diff --git a/.gitlab/merge_request_templates/Default.md b/.gitlab/merge_request_templates/Default.md
index a82cb3f8..2acf3b1e 100644
--- a/.gitlab/merge_request_templates/Default.md
+++ b/.gitlab/merge_request_templates/Default.md
@@ -27,3 +27,4 @@ This checklist encourages us to confirm any changes have been analyzed to reduce
 * [ ] Have you resolved any lint issues?
 * [ ] Have you assigned a reviewer?
 * [ ] Have you applied correct labels?
+* [ ] Have you updated CHANGELOG.md?
diff --git a/.gitlab/release.yml b/.gitlab/release.yml
new file mode 100644
index 00000000..a54dce02
--- /dev/null
+++ b/.gitlab/release.yml
@@ -0,0 +1,16 @@
+release:
+  stage: release
+  image: registry.gitlab.com/gitlab-org/release-cli:latest
+  rules:
+    - if: $CI_COMMIT_TAG
+  script:
+    - echo "Running release job for tag $CI_COMMIT_TAG"
+  release:
+    tag_name: $CI_COMMIT_TAG
+    name: Crafty $CI_COMMIT_TAG
+    description: ./CHANGELOG.md
+    assets:
+      links:
+        - name: Windows Package
+          url: "https://gitlab.com/some/repo/-/jobs/$(cat CI_JOB_ID.txt)/artifacts/download"
+          link_type: package
diff --git a/.gitlab/windows-build.yml b/.gitlab/windows-build.yml
index 1487155f..277710f0 100644
--- a/.gitlab/windows-build.yml
+++ b/.gitlab/windows-build.yml
@@ -50,9 +50,11 @@ win-prod-build:
       - .venv/
   rules:
     - if: "$CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH"
+    - if: $CI_COMMIT_TAG
   environment:
     name: production
   script:
+    - echo "${CI_JOB_ID}" > CI_JOB_ID.txt
     - |
       $ErrorActionPreference = "Stop"
       py -m venv .venv
@@ -79,5 +81,7 @@ win-prod-build:
     paths:
       - app\
       - .\crafty_commander.exe
+      - CI_JOB_ID.txt
+    expire_in: never
     exclude:
       - app\classes\**\*
diff --git a/CHANGELOG.md b/CHANGELOG.md
new file mode 100644
index 00000000..831e44e8
--- /dev/null
+++ b/CHANGELOG.md
@@ -0,0 +1,31 @@
+# Changelog
+
+## [4.0.2] - 2022/06/16
+
+### New features
+ None
+
+### Bug fixes
+- Fix winreg import pass on non-NT systems ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/344))
+- Make the WebSocket automatically reconnect. ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/345))
+- Fix an error when there are no servers ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/346))
+- Use relative paths for the jarfile and logs ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/347))
+- Flatten all instances of username creation or editing, usernames should be lower case.
+- - ([Merge Request 1](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/342))
+- - ([Merge Request 2](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/351))
+- Add version inheretence & config check ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/353))
+- Fix support log temp file deletion issue/hang ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/354))
+
+## [4.0.1] - 2022/06/15
+
+### New features
+ None
+
+### Bug fixes
+
+- Remove session.lock warning ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/338))
+- Correct Dutch Spacing Issue ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/340))
+- Remove no-else-* pylint exemptions and tidy code. ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/342))
+- Make unRAID more readable, and flatten path to lower, to fit standard practice. ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/337))
+- Fix Java Pathing issues on windows ([Commit](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/343/diffs?commit_id=cda2120579083d447db5dbeb5489822880f4cae7))
+

From 9eb560845c539b7bb00144b7d0b814626f45c0d4 Mon Sep 17 00:00:00 2001
From: Zedifus <zedifus@zediverse.gg>
Date: Thu, 16 Jun 2022 18:43:06 +0100
Subject: [PATCH 12/13] Silence lint on release pipe, add (---)

---
 .gitlab/release.yml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/.gitlab/release.yml b/.gitlab/release.yml
index a54dce02..10f543d4 100644
--- a/.gitlab/release.yml
+++ b/.gitlab/release.yml
@@ -1,3 +1,5 @@
+# yamllint disable rule:line-length
+---
 release:
   stage: release
   image: registry.gitlab.com/gitlab-org/release-cli:latest

From e47cad13f998e9a64001865238d957452ca1abf2 Mon Sep 17 00:00:00 2001
From: Zedifus <zedifus@zediverse.gg>
Date: Thu, 16 Jun 2022 18:45:07 +0100
Subject: [PATCH 13/13] Bump version

---
 README.md               | 4 ++--
 app/config/version.json | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index 2b1e2b3d..7334e0e1 100644
--- a/README.md
+++ b/README.md
@@ -2,11 +2,11 @@
 
 [![Code style: black](https://img.shields.io/badge/code%20style-black-000000.svg)](https://github.com/psf/black)
 [![Supported Python Versions](https://shields.io/badge/python-3.8%20%7C%203.9%20%7C%203.10%20-blue)](https://www.python.org)
-[![Version(temp-hardcoded)](https://img.shields.io/badge/release-v4.0.1--beta-orange)](https://gitlab.com/crafty-controller/crafty-4)
+[![Version(temp-hardcoded)](https://img.shields.io/badge/release-v4.0.2--beta-orange)](https://gitlab.com/crafty-controller/crafty-4/-/releases)
 [![Code Quality(temp-hardcoded)](https://img.shields.io/badge/code%20quality-10-brightgreen)](https://gitlab.com/crafty-controller/crafty-4)
 [![Build Status](https://gitlab.com/crafty-controller/crafty-4/badges/master/pipeline.svg)](https://gitlab.com/crafty-controller/crafty-4/-/commits/master)
 
-# Crafty Controller 4.0.1-beta
+# Crafty Controller 4.0.2-beta
 > Python based Control Panel for your Minecraft Server
 
 ## What is Crafty Controller?
diff --git a/app/config/version.json b/app/config/version.json
index 1b3a7c55..4b9d8e69 100644
--- a/app/config/version.json
+++ b/app/config/version.json
@@ -1,6 +1,6 @@
 {
   "major": 4,
   "minor": 0,
-  "sub": 1,
+  "sub": 2,
   "meta": "beta"
 }