From d1b817443c5b34e4b564e31e9dc02e54d8db6188 Mon Sep 17 00:00:00 2001
From: Richard Underwood <richard.underwood@digitaslbi.com>
Date: Wed, 24 Aug 2016 11:32:43 +0100
Subject: [PATCH] Initial implementation of log rotation.

---
 includes/config.inc.php-dist |  5 +++++
 includes/misc.inc.php        | 31 +++++++++++++++++++++++++++++++
 index.php                    | 29 +++++++++++++++++++++++++++++
 logs.php                     | 13 ++++++++++++-
 4 files changed, 77 insertions(+), 1 deletion(-)

diff --git a/includes/config.inc.php-dist b/includes/config.inc.php-dist
index 51f0bdd..bc00616 100644
--- a/includes/config.inc.php-dist
+++ b/includes/config.inc.php-dist
@@ -7,7 +7,12 @@ $apiproto      = 'http'; # http | https
 $apisslverify  = FALSE;  # Verify SSL Certificate if using https for apiproto
 $allowzoneadd  = FALSE;  # Allow normal users to add zones
 $logging = TRUE;
+$allowclearlogs = TRUE;  # Allow clearing of log entries
+$allowrotatelogs = FALSE;# Allow rotation to text file on server
 
+# Log directory - if allowrotatelogs is set, this is where the logs will
+# be written. It must be writeable by the web server user.
+$logsdirectory = "../etc";
 
 # If you configure this, nsedit will try to authenticate via WeFact too.
 # Debtors will be added to the sqlitedatabase with their crypted password.
diff --git a/includes/misc.inc.php b/includes/misc.inc.php
index 7041d0f..4c88c01 100644
--- a/includes/misc.inc.php
+++ b/includes/misc.inc.php
@@ -277,6 +277,37 @@ function clearlogs() {
     writelog("Logtable truncated.");
 }
 
+function rotatelogs() {
+    global $logging, $logsdirectory;
+    if ($logging !== TRUE)
+        return FALSE;
+
+    if(!is_dir($logsdirectory) || !is_writable($logsdirectory)) {
+      writelog("Logs directory cannot be written to.");
+      return FALSE;
+    }
+
+    date_default_timezone_set('UTC');
+    $filename = date("Y-m-d-His") . ".json";
+    $file = fopen($logsdirectory . "/" . $filename, "x");
+
+    if($file === FALSE) {
+      writelog("Can't create file for log rotation.");
+      return FALSE;
+    }
+
+    if(fwrite($file,json_encode(getlogs())) === FALSE) {
+        writelog("Can't write to file for log rotation.");
+        fclose($file);
+        return FALSE;
+    } else {
+        fclose($file);
+        clearlogs();
+        return $filename;
+    }
+
+}
+
 function writelog($line) {
     global $logging;
     if ($logging !== TRUE)
diff --git a/index.php b/index.php
index 78d1ee9..eb4cfbd 100644
--- a/index.php
+++ b/index.php
@@ -116,6 +116,9 @@ if ($blocklogin === TRUE) {
     <div id="clearlogs" style="display: none;">
         Are you sure you want to clear all the logs? Maybe save them first?
     </div>
+    <div id="rotatelogs" style="display: none;">
+        Are you sure you want to rotate the logs?
+    </div>
     <div id="searchlogs" style="display: none; text-align: right;">
         <table border="0">
         <tr><td>User:</td><td><input type="text" id ="searchlogs-user"><br></td></tr>
@@ -1043,6 +1046,31 @@ $(document).ready(function () {
                         });
                     }
                 },
+                <?php if($allowrotatelogs === TRUE) { ?>
+                {
+                    icon: 'img/export.png',
+                    text: 'Rotate logs',
+                    click: function() {
+                        $("#rotatelogs").dialog({
+                            modal: true,
+                            title: "Rotate logs",
+                            width: 'auto',
+                            buttons: {
+                                Ok: function() {
+                                    $.get("logs.php?action=rotate");
+                                    $( this ).dialog( "close" );
+                                    $('#Logs').jtable('load');
+                                },
+                                Cancel: function() {
+                                    $( this ).dialog( "close" );
+                                    return false;
+                                }
+                            }
+                        });
+                    }
+                },
+                <?php } ?>
+                <?php if($allowclearlogs === TRUE) { ?>
                 {
                     icon: 'img/delete_inverted.png',
                     text: 'Clear logs',
@@ -1065,6 +1093,7 @@ $(document).ready(function () {
                         });
                     }
                 },
+                <?php } ?>
                 {
                     icon: 'img/export.png',
                     text: 'Save logs',
diff --git a/logs.php b/logs.php
index 1f4d32f..20a227f 100644
--- a/logs.php
+++ b/logs.php
@@ -61,7 +61,18 @@ case "export":
     break;
 
 case "clear":
-    clearlogs();
+    if($allowclearlogs === TRUE) {
+        clearlogs();
+    } else {
+        jtable_respond(null, 'error', 'Invalid action');
+    }
+    break;
+case "rotate":
+    if($allowrotatelogs === TRUE) {
+        rotatelogs();
+    } else {
+        jtable_respond(null, 'error', 'Invalid action');
+    }
     break;
 default:
     jtable_respond(null, 'error', 'Invalid action');