Merge pull request #121 from woogerboy21/serverlog-filtering

Server log filtering

servatrice/servatrice.ini.example

@@ -5,6 +5,8 @@ logfile=server.log
 name="My Cockatrice server"
 id=1
 number_pools=1
+writelog=1
+logfilters=""
 
 [servernetwork]
 active=0

servatrice/src/server_logger.cpp

@@ -3,6 +3,7 @@
 #include <QFile>
 #include <QTextStream>
 #include <QDateTime>
+#include <QSettings>
 #include <iostream>
 #ifdef Q_OS_UNIX
 # include <sys/types.h>
@@ -11,99 +12,122 @@
 #endif
 
 ServerLogger::ServerLogger(bool _logToConsole, QObject *parent)
-	: QObject(parent), logToConsole(_logToConsole), flushRunning(false)
+    : QObject(parent), logToConsole(_logToConsole), flushRunning(false)
 {
 }
 
 ServerLogger::~ServerLogger()
 {
-	flushBuffer();
+    flushBuffer();
-	// This does not work with the destroyed() signal as this destructor is called after the main event loop is done.
+    // This does not work with the destroyed() signal as this destructor is called after the main event loop is done.
-	thread()->quit();
+    thread()->quit();
 }
 
 void ServerLogger::startLog(const QString &logFileName)
 {
-	if (!logFileName.isEmpty()) {
+    if (!logFileName.isEmpty()) {
-		logFile = new QFile("server.log", this);
+        logFile = new QFile("server.log", this);
-		logFile->open(QIODevice::Append);
+        logFile->open(QIODevice::Append);
 #ifdef Q_OS_UNIX
-		::socketpair(AF_UNIX, SOCK_STREAM, 0, sigHupFD);
+        ::socketpair(AF_UNIX, SOCK_STREAM, 0, sigHupFD);
 
-		snHup = new QSocketNotifier(sigHupFD[1], QSocketNotifier::Read, this);
+        snHup = new QSocketNotifier(sigHupFD[1], QSocketNotifier::Read, this);
-		connect(snHup, SIGNAL(activated(int)), this, SLOT(handleSigHup()));
+        connect(snHup, SIGNAL(activated(int)), this, SLOT(handleSigHup()));
 #endif
-	} else
+    } else
-		logFile = 0;
+        logFile = 0;
-	
+    
-	connect(this, SIGNAL(sigFlushBuffer()), this, SLOT(flushBuffer()), Qt::QueuedConnection);
+    connect(this, SIGNAL(sigFlushBuffer()), this, SLOT(flushBuffer()), Qt::QueuedConnection);
 }
 
 void ServerLogger::logMessage(QString message, void *caller)
 {
-	if (!logFile)
+    if (!logFile)
-		return;
+        return;
-	
+    
-	bufferMutex.lock();
+    QString callerString;
-	QString callerString;
+    if (caller)
-	if (caller)
+        callerString = QString::number((qulonglong) caller, 16) + " ";
-		callerString = QString::number((qulonglong) caller, 16) + " ";
+        
-	buffer.append(QDateTime::currentDateTime().toString() + " " + callerString + message);
+    //filter out all log entries based on values in configuration file
-	bufferMutex.unlock();
+    QSettings *settings = new QSettings("servatrice.ini", QSettings::IniFormat);
-	
+    bool shouldWeWriteLog = settings->value("server/writelog").toBool();
-	emit sigFlushBuffer();
+    QString logFilters = settings->value("server/logfilters").toString();
+    QStringList listlogFilters = logFilters.split(",", QString::SkipEmptyParts); 
+    bool shouldWeSkipLine = false; 
+    
+    if (!shouldWeWriteLog)
+        return;
+
+    if (!logFilters.trimmed().isEmpty()){
+        shouldWeSkipLine = true;
+        foreach(QString logFilter, listlogFilters){ 
+            if (message.contains(logFilter, Qt::CaseInsensitive)){
+                shouldWeSkipLine = false;
+                break;
+            }
+        }
+    }
+
+    if (shouldWeSkipLine)
+        return;
+
+    bufferMutex.lock();
+    buffer.append(QDateTime::currentDateTime().toString() + " " + callerString + message);
+    bufferMutex.unlock();
+    emit sigFlushBuffer();
 }
 
 void ServerLogger::flushBuffer()
 {
-	if (flushRunning)
+    if (flushRunning)
-		return;
+        return;
-	
+    
-	flushRunning = true;
+    flushRunning = true;
-	QTextStream stream(logFile);
+    QTextStream stream(logFile);
-	forever {
+    forever {
-		bufferMutex.lock();
+        bufferMutex.lock();
-		if (buffer.isEmpty()) {
+        if (buffer.isEmpty()) {
-			bufferMutex.unlock();
+            bufferMutex.unlock();
-			flushRunning = false;
+            flushRunning = false;
-			return;
+            return;
-		}
+        }
-		QString message = buffer.takeFirst();
+        QString message = buffer.takeFirst();
-		bufferMutex.unlock();
+        bufferMutex.unlock();
-		
+        
-		stream << message << "\n";
+        stream << message << "\n";
-		stream.flush();
+        stream.flush();
-		
+        
-		if (logToConsole)
+        if (logToConsole)
-			std::cout << message.toStdString() << std::endl;
+            std::cout << message.toStdString() << std::endl;
-	}
+    }
 }
 
 void ServerLogger::hupSignalHandler(int /*unused*/)
 {
 #ifdef Q_OS_UNIX
-	if (!logFile)
+    if (!logFile)
-		return;
+        return;
-	
+    
-	char a = 1;
+    char a = 1;
-	::write(sigHupFD[0], &a, sizeof(a));
+    ::write(sigHupFD[0], &a, sizeof(a));
 #endif
 }
 
 void ServerLogger::handleSigHup()
 {
 #ifdef Q_OS_UNIX
-	if (!logFile)
+    if (!logFile)
-		return;
+        return;
-	
+    
-	snHup->setEnabled(false);
+    snHup->setEnabled(false);
-	char tmp;
+    char tmp;
-	::read(sigHupFD[1], &tmp, sizeof(tmp));
+    ::read(sigHupFD[1], &tmp, sizeof(tmp));
-	
+    
-	logFile->close();
+    logFile->close();
-	logFile->open(QIODevice::Append);
+    logFile->open(QIODevice::Append);
-	
+    
-	snHup->setEnabled(true);
+    snHup->setEnabled(true);
 #endif
 }