]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/logger.cpp
Remove InspIRCd* parameters and fields
[user/henk/code/inspircd.git] / src / logger.cpp
index c9171292a646958e38ce8aefb2d6600ba50a3437..e2baa4b8da4137b0434579b8b8a68a40a77865ad 100644 (file)
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
+ * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
@@ -11,7 +11,7 @@
  * ---------------------------------------------------
  */
 
-/* $Core: libIRCDlogger */
+/* $Core */
 
 #include "inspircd.h"
 
@@ -31,7 +31,7 @@
  *
  * How it works:
  *  Modules create their own logstream types (core will create one for 'file logging' for example) and create instances of these logstream types
- *  and register interest in a certain logtype. Globbing is not here, with the exception of * - for all events.. loglevel is used to drop 
+ *  and register interest in a certain logtype. Globbing is not here, with the exception of * - for all events.. loglevel is used to drop
  *  events that are of no interest to a logstream.
  *
  *  When Log is called, the vector of logstreams for that type is iterated (along with the special vector for "*"), and all registered logstreams
  * NOTE: Somehow we have to let LogManager manage the non-blocking file streams and provide an interface to share them with various LogStreams,
  *       as, for example, a user may want to let 'KILL' and 'XLINE' snotices go to /home/ircd/inspircd/logs/operactions.log, or whatever. How
  *       can we accomplish this easily? I guess with a map of pre-loved logpaths, and a pointer of FILE *..
- * 
+ *
  */
 
 void LogManager::SetupNoFork()
 {
        if (!noforkstream)
        {
-               FileWriter* fw = new FileWriter(ServerInstance, stdout);
-               noforkstream = new FileLogStream(ServerInstance, ServerInstance->Config->forcedebug ? DEBUG : ServerInstance->Config->LogLevel, fw);
+               FileWriter* fw = new FileWriter(stdout);
+               noforkstream = new FileLogStream(ServerInstance->Config->forcedebug ? DEBUG : DEFAULT, fw);
        }
        else
        {
-               noforkstream->ChangeLevel(ServerInstance->Config->forcedebug ? DEBUG : ServerInstance->Config->LogLevel);
+               noforkstream->ChangeLevel(ServerInstance->Config->forcedebug ? DEBUG : DEFAULT);
        }
        AddLogType("*", noforkstream, false);
 }
 
 void LogManager::OpenFileLogs()
 {
-       if (ServerInstance->Config->nofork) SetupNoFork(); // Call this to reregister the nofork stream.
-       if (!ServerInstance->Config->writelog) return; // Skip rest of logfile opening if we are running -nolog.
-       ConfigReader* Conf = new ConfigReader(ServerInstance);
+       /* Re-register the nofork stream if necessary. */
+       if (ServerInstance->Config->nofork)
+       {
+               SetupNoFork();
+       }
+       /* Skip rest of logfile opening if we are running -nolog. */
+       if (!ServerInstance->Config->writelog)
+       {
+               return;
+       }
+       ConfigReader* Conf = new ConfigReader;
        std::map<std::string, FileWriter*> logmap;
        std::map<std::string, FileWriter*>::iterator i;
        for (int index = 0; index < Conf->Enumerate("log"); ++index)
        {
                std::string method = Conf->ReadValue("log", "method", index);
-               if (method != "file") continue;
+               if (method != "file")
+               {
+                       continue;
+               }
                std::string type = Conf->ReadValue("log", "type", index);
                std::string level = Conf->ReadValue("log", "level", index);
                int loglevel = DEFAULT;
@@ -98,27 +109,22 @@ void LogManager::OpenFileLogs()
                if ((i = logmap.find(target)) == logmap.end())
                {
                        FILE* f = fopen(target.c_str(), "a");
-                       fw = new FileWriter(ServerInstance, f);
+                       fw = new FileWriter(f);
                        logmap.insert(std::make_pair(target, fw));
                }
                else
                {
                        fw = i->second;
                }
-               FileLogStream* fls = new FileLogStream(ServerInstance, loglevel, fw);
-               irc::spacesepstream css(type);
-               std::string tok;
-               while (css.GetToken(tok))
-               {
-                       AddLogType(tok, fls, true);
-               }
+               FileLogStream* fls = new FileLogStream(loglevel, fw);
+               AddLogTypes(type, fls, true);
        }
 }
 
 void LogManager::CloseLogs()
 {
        std::map<std::string, std::vector<LogStream*> >().swap(LogStreams); /* Clear it */
-       std::vector<LogStream*>().swap(GlobalLogStreams); /* Clear it */
+       std::map<LogStream*, std::vector<std::string> >().swap(GlobalLogStreams); /* Clear it */
        for (std::map<LogStream*, int>::iterator i = AllLogStreams.begin(); i != AllLogStreams.end(); ++i)
        {
                delete i->first;
@@ -126,12 +132,55 @@ void LogManager::CloseLogs()
        std::map<LogStream*, int>().swap(AllLogStreams); /* And clear it */
 }
 
+void LogManager::AddLogTypes(const std::string &types, LogStream* l, bool autoclose)
+{
+       irc::spacesepstream css(types);
+       std::string tok;
+       std::vector<std::string> excludes;
+       while (css.GetToken(tok))
+       {
+               if (tok.empty())
+               {
+                       continue;
+               }
+               if (tok.at(0) == '-')
+               {
+                       /* Exclude! */
+                       excludes.push_back(tok.substr(1));
+               }
+               else
+               {
+                       AddLogType(tok, l, autoclose);
+               }
+       }
+       // Handle doing things like: USERINPUT USEROUTPUT -USERINPUT should be the same as saying just USEROUTPUT.
+       // (This is so modules could, for example, inject exclusions for logtypes they can't handle.)
+       for (std::vector<std::string>::iterator i = excludes.begin(); i != excludes.end(); ++i)
+       {
+               if (*i == "*")
+               {
+                       /* -* == Exclude all. Why someone would do this, I dunno. */
+                       DelLogStream(l);
+                       return;
+               }
+               DelLogType(*i, l);
+       }
+       // Now if it's registered as a global, add the exclusions there too.
+       std::map<LogStream *, std::vector<std::string> >::iterator gi = GlobalLogStreams.find(l);
+       if (gi != GlobalLogStreams.end())
+       {
+               gi->second.swap(excludes); // Swap with the vector in the hash.
+       }
+}
+
 bool LogManager::AddLogType(const std::string &type, LogStream *l, bool autoclose)
 {
        std::map<std::string, std::vector<LogStream *> >::iterator i = LogStreams.find(type);
 
        if (i != LogStreams.end())
+       {
                i->second.push_back(l);
+       }
        else
        {
                std::vector<LogStream *> v;
@@ -140,7 +189,9 @@ bool LogManager::AddLogType(const std::string &type, LogStream *l, bool autoclos
        }
 
        if (type == "*")
-               GlobalLogStreams.push_back(l);
+       {
+               GlobalLogStreams.insert(std::make_pair(l, std::vector<std::string>()));
+       }
 
        if (autoclose)
        {
@@ -165,14 +216,21 @@ void LogManager::DelLogStream(LogStream* l)
                std::vector<LogStream*>::iterator it;
                while ((it = std::find(i->second.begin(), i->second.end(), l)) != i->second.end())
                {
-                       if (it == i->second.end()) continue;
+                       if (it == i->second.end())
+                               continue;
                        i->second.erase(it);
                }
        }
-       std::vector<LogStream *>::iterator gi = std::find(GlobalLogStreams.begin(), GlobalLogStreams.end(), l);
-       if (gi != GlobalLogStreams.end()) GlobalLogStreams.erase(gi);
+       std::map<LogStream *, std::vector<std::string> >::iterator gi = GlobalLogStreams.find(l);
+       if (gi != GlobalLogStreams.end())
+       {
+               GlobalLogStreams.erase(gi);
+       }
        std::map<LogStream*, int>::iterator ai = AllLogStreams.begin();
-       if (ai == AllLogStreams.end()) return; /* Done. */
+       if (ai == AllLogStreams.end())
+       {
+               return; /* Done. */
+       }
        delete ai->first;
        AllLogStreams.erase(ai);
 }
@@ -182,7 +240,7 @@ bool LogManager::DelLogType(const std::string &type, LogStream *l)
        std::map<std::string, std::vector<LogStream *> >::iterator i = LogStreams.find(type);
        if (type == "*")
        {
-               std::vector<LogStream *>::iterator gi = std::find(GlobalLogStreams.begin(), GlobalLogStreams.end(), l);
+               std::map<LogStream *, std::vector<std::string> >::iterator gi = GlobalLogStreams.find(l);
                if (gi != GlobalLogStreams.end()) GlobalLogStreams.erase(gi);
        }
 
@@ -209,7 +267,10 @@ bool LogManager::DelLogType(const std::string &type, LogStream *l)
        }
 
        std::map<LogStream*, int>::iterator ai = AllLogStreams.find(l);
-       if (ai == AllLogStreams.end()) return true;
+       if (ai == AllLogStreams.end())
+       {
+               return true;
+       }
 
        if ((--ai->second) < 1)
        {
@@ -223,7 +284,9 @@ bool LogManager::DelLogType(const std::string &type, LogStream *l)
 void LogManager::Log(const std::string &type, int loglevel, const char *fmt, ...)
 {
        if (Logging)
+       {
                return;
+       }
 
        va_list a;
        static char buf[65536];
@@ -238,28 +301,28 @@ void LogManager::Log(const std::string &type, int loglevel, const char *fmt, ...
 void LogManager::Log(const std::string &type, int loglevel, const std::string &msg)
 {
        if (Logging)
+       {
                return;
+       }
 
        Logging = true;
 
-       std::vector<LogStream *>::iterator gi = GlobalLogStreams.begin();
-
-       while (gi != GlobalLogStreams.end())
+       for (std::map<LogStream *, std::vector<std::string> >::iterator gi = GlobalLogStreams.begin(); gi != GlobalLogStreams.end(); ++gi)
        {
-               (*gi)->OnLog(loglevel, type, msg);
-               gi++;
+               if (std::find(gi->second.begin(), gi->second.end(), type) != gi->second.end())
+               {
+                       continue;
+               }
+               gi->first->OnLog(loglevel, type, msg);
        }
 
        std::map<std::string, std::vector<LogStream *> >::iterator i = LogStreams.find(type);
 
        if (i != LogStreams.end())
        {
-               std::vector<LogStream *>::iterator it = i->second.begin();
-
-               while (it != i->second.end())
+               for (std::vector<LogStream *>::iterator it = i->second.begin(); it != i->second.end(); ++it)
                {
                        (*it)->OnLog(loglevel, type, msg);
-                       it++;
                }
        }
 
@@ -267,60 +330,26 @@ void LogManager::Log(const std::string &type, int loglevel, const std::string &m
 }
 
 
-FileWriter::FileWriter(InspIRCd* Instance, FILE* logfile)
-: ServerInstance(Instance), log(logfile), writeops(0)
+FileWriter::FileWriter(FILE* logfile)
+: log(logfile), writeops(0)
 {
-       if (log)
-       {
-               Instance->SE->NonBlocking(fileno(log));
-               SetFd(fileno(log));
-               buffer.clear();
-       }
 }
 
-bool FileWriter::Readable()
+void FileWriter::HandleEvent(EventType ev, int)
 {
-       return false;
-}
-    
-void FileWriter::HandleEvent(EventType, int)
-{
-       WriteLogLine("");
-       if (log)
-               ServerInstance->SE->DelFd(this);
 }
 
 void FileWriter::WriteLogLine(const std::string &line)
 {
-       if (line.length())
-               buffer.append(line);
+       if (log == NULL)
+               return;
+// XXX: For now, just return. Don't throw an exception. It'd be nice to find out if this is happening, but I'm terrified of breaking so close to final release. -- w00t
+//             throw CoreException("FileWriter::WriteLogLine called with a closed logfile");
 
-       if (log)
+       fprintf(log,"%s",line.c_str());
+       if (writeops++ % 20)
        {
-               int written = fprintf(log,"%s",buffer.c_str());
-#ifdef WINDOWS
-               buffer.clear();
-#else
-               if ((written >= 0) && (written < (int)buffer.length()))
-               {
-                       buffer.erase(0, buffer.length());
-                       ServerInstance->SE->AddFd(this);
-               }
-               else if (written == -1)
-               {
-                       if (errno == EAGAIN)
-                               ServerInstance->SE->AddFd(this);
-               }
-               else
-               {
-                       /* Wrote the whole buffer, and no need for write callback */
-                       buffer.clear();
-               }
-#endif
-               if (writeops++ % 20)
-               {
-                       fflush(log);
-               }
+               fflush(log);
        }
 }
 
@@ -328,25 +357,13 @@ void FileWriter::Close()
 {
        if (log)
        {
-               ServerInstance->SE->Blocking(fileno(log));
-
-               if (buffer.size())
-                       fprintf(log,"%s",buffer.c_str());
-
-#ifndef WINDOWS
-               ServerInstance->SE->DelFd(this);
-#endif
-
                fflush(log);
                fclose(log);
+               log = NULL;
        }
-
-       buffer.clear();
 }
 
 FileWriter::~FileWriter()
 {
        this->Close();
 }
-
-