Quite a few fixes in the CP, logs now match the rest

cp/app/Controllers/Auth/AuthController.php

@@ -32,8 +32,8 @@ class AuthController extends Controller
      * @throws \DI\NotFoundException
      */
     public function createLogin(Request $request, Response $response){
-        $isWebAuthnEnabled = envi('WEB_AUTHN_ENABLED') === 'true';
+        $isWebAuthnEnabled = (envi('WEB_AUTHN_ENABLED') === 'true') ? true : false;
-        return view($response, 'auth/login.twig', ['isWebaEnabled' => $isWebaEnabled]);
+        return view($response, 'auth/login.twig', ['isWebaEnabled' => $isWebAuthnEnabled]);
     }
 
     /**

cp/app/Controllers/DomainsController.php

@@ -2046,6 +2046,7 @@ class DomainsController extends Controller
                     [ $domain_id ]
                 );
 
+                if (is_array($results) && !empty($results)) {
                     foreach ($results as $row) {
                         $status = $row['status'];
                         if (preg_match('/.*(UpdateProhibited|DeleteProhibited)$/', $status) || preg_match('/^pending/', $status)) {
@@ -2053,6 +2054,7 @@ class DomainsController extends Controller
                             return $response->withHeader('Location', '/domains')->withStatus(302);
                         }
                     }
+                }
 
                 $grace_period = 30;
                 
@@ -2112,6 +2114,7 @@ class DomainsController extends Controller
                                     [$domain_id]
                                 );
 
+                                if (is_array($hostIds) && !empty($hostIds)) {
                                     foreach ($hostIds as $host) {
                                         $host_id = $host['id'];
 
@@ -2135,6 +2138,7 @@ class DomainsController extends Controller
                                             ]
                                         );
                                     }
+                                }
 
                                 // Delete domain related records
                                 $db->delete(

cp/app/Controllers/HomeController.php

@@ -188,7 +188,7 @@ class HomeController extends Controller
 
     public function mode(Request $request, Response $response)
     {
-        if ($_SESSION['_screen_mode'] == 'dark') {
+        if (isset($_SESSION['_screen_mode']) && $_SESSION['_screen_mode'] == 'dark') {
             $_SESSION['_screen_mode'] = 'light';
         } else {
             $_SESSION['_screen_mode'] = 'dark';

cp/app/Controllers/SystemController.php

@@ -692,6 +692,7 @@ class SystemController extends Controller
                     return $response->withHeader('Location', '/registry/tld/'.$tld_extension)->withStatus(302);
                 }
 
+                if (isset($data['dnssec_enable'], $data['bind9_enable'])) {
                     if ($data['dnssec_enable'] === 'on' && $data['bind9_enable'] === 'on') {
                         $dnssec_both = 1;
                     } elseif ($data['dnssec_enable'] === 'on' && $data['bind9_enable'] !== 'on') {
@@ -701,6 +702,7 @@ class SystemController extends Controller
                         $this->container->get('flash')->addMessage('error', 'DNSSEC can be only enabled for BIND9');
                         return $response->withHeader('Location', '/registry/tld/'.$tld_extension)->withStatus(302);
                     }
+                }
 
                 try {
                     $db->beginTransaction();

cp/app/Lib/Logger.php

@@ -3,14 +3,12 @@
 namespace App\Lib;
 
 use Monolog\ErrorHandler;
-use Monolog\Handler\StreamHandler;
+use Monolog\Handler\RotatingFileHandler;
-
 use Whoops\Handler\PrettyPageHandler;
 use Whoops\Run;
+
 /**
  * Logger
- *
- * @author    Hezekiah O. <support@hezecom.com>
  */
 class Logger extends \Monolog\Logger
 {
@@ -26,14 +24,20 @@ class Logger extends \Monolog\Logger
     {
         parent::__construct($key);
 
-        if (empty($config)) {
+        // Set log path and maximum number of files to retain
         $LOG_PATH = '/var/log/namingo';
+        $maxFiles = 30; // Number of days to keep logs
+
+        if (empty($config)) {
             $config = [
-                'logFile' => "{$LOG_PATH}/{$key}.log",
+                'logFile' => "{$LOG_PATH}/cp.log", // Base log name
-                'logLevel' => \Monolog\Logger::DEBUG
+                'logLevel' => \Monolog\Logger::DEBUG,
+                'maxFiles' => $maxFiles,
             ];
         }
-        $this->pushHandler(new StreamHandler($config['logFile'], $config['logLevel']));
+
+        // Use RotatingFileHandler for automatic rotation
+        $this->pushHandler(new RotatingFileHandler($config['logFile'], $config['maxFiles'], $config['logLevel']));
     }
 
     /**
@@ -51,31 +55,27 @@ class Logger extends \Monolog\Logger
     }
 
     /**
-     * Output error bate on environment
+     * Output error based on environment
      */
     public static function systemLogs($enable = true)
     {
-
-        $LOG_PATH = '/var/log/namingo';
-        $appEnv = envi('APP_ENV') ?? 'local';
-
         if ($enable) {
-            // output pretty html error
+            // Output pretty HTML error
             self::htmlError();
         } else {
-            // Error Log to file
+            // Register system errors to rotating log file
-            self::$loggers['error'] = new Logger('errors');
+            $logger = new Logger('errors'); // Key 'errors' remains for compatibility
-            self::$loggers['error']->pushHandler(new StreamHandler("{$LOG_PATH}/cp.log"));
+            ErrorHandler::register($logger);
-            ErrorHandler::register(self::$loggers['error']);
         }
     }
 
     /**
-     * Display pretty html formatted errors during development
+     * Display pretty HTML formatted errors during development
      */
-    public static function htmlError(){
+    public static function htmlError()
-        $run = new Run;
+    {
-        $run->pushHandler(new PrettyPageHandler);
+        $run = new Run();
+        $run->pushHandler(new PrettyPageHandler());
         $run->register();
     }
 }