fixed endless redirect issue on non-existant paths

app/classes/minecraft/controller.py

@@ -27,7 +27,6 @@ class Controller:
             self.servers_list = []
 
         servers = db_helper.get_all_defined_servers()
-        print(servers)
 
         for s in servers:
             settings_file = os.path.join(s['path'], 'server.properties')

app/classes/web/default_handler.py

@@ -0,0 +1,23 @@
+import logging
+
+from app.classes.web.base_handler import BaseHandler
+
+logger = logging.getLogger(__name__)
+
+
+class DefaultHandler(BaseHandler):
+
+    def get(self, page=None):
+
+        # sensible defaults
+        template = "public/404.html"
+
+        self.render(template)
+
+    def post(self, page=None):
+
+        # sensible defaults
+        template = "public/404.html"
+
+        self.render(template)
+

app/classes/web/panel_handler.py

@@ -35,8 +35,12 @@ class PanelHandler(BaseHandler):
             'hosts_data': db_helper.get_latest_hosts_stats()
 
         }
-        print(page_data['hosts_data'])
+        # print(page_data['hosts_data'])
 
+        # if no servers defined, let's go to the build server area
+        if page_data['server_stats']['total'] == 0:
+            self.redirect("server/step1")
+            return False
 
         if page == 'unauthorized':
             template = "panel/denied.html"

app/classes/web/public_handler.py

@@ -60,7 +60,7 @@ class PublicHandler(BaseHandler):
 
         # if we have no page, let's go to login
         else:
-            self.redirect("public/login")
+            self.redirect("/public/login")
 
         self.render(template, data=page_data)
 

app/classes/web/tornado.py

@@ -20,6 +20,7 @@ try:
     import tornado.httpserver
     from app.classes.web.public_handler import PublicHandler
     from app.classes.web.panel_handler import PanelHandler
+    from app.classes.web.default_handler import DefaultHandler
 
 except ModuleNotFoundError as e:
     logger.critical("Import Error: Unable to load {} module".format(e, e.name))
@@ -117,7 +118,7 @@ class webserver:
         tornado.locale.set_default_locale(lang)
 
         handlers = [
-            (r'/', PublicHandler),
+            (r'/', DefaultHandler),
             (r'/public/(.*)', PublicHandler),
             (r'/panel/(.*)', PanelHandler),
             ]

main.py

@@ -97,8 +97,6 @@ if __name__ == '__main__':
         installer.create_tables()
         installer.default_settings()
 
-    # installer.create_tables()
-
     # init servers
     logger.info("Initializing all servers defined")
     console.info("Initializing all servers defined")