Chain ValidateServerName onto ValidateHostname so that the servername gets hostname...

[user/henk/code/inspircd.git] / src / modules / m_chanlog.cpp

diff --git a/src/modules/m_chanlog.cpp b/src/modules/m_chanlog.cpp
index 4a8a41cb0c2489f4b9a3ac52289f590748073a3b..5070f190a7cac59b4b70fc31e57f05891ba07c6c 100644 (file)
--- a/src/modules/m_chanlog.cpp
+++ b/src/modules/m_chanlog.cpp
@@ -26,18 +26,23 @@ class ChannelLogStream : public LogStream
        virtual void OnLog(int loglevel, const std::string &type, const std::string &msg)
        {
                Channel *c = ServerInstance->FindChan(channel);
+               static bool Logging = false;
 
-printf("I got called\n");
                if (loglevel < this->loglvl)
                        return;
 
+               if (Logging)
+                       return;
+
                if (c)
                {
+                       Logging = true; // this avoids (rare chance) loops with logging server IO on networks
                        char buf[MAXBUF];
                        snprintf(buf, MAXBUF, "\2%s\2: %s", type.c_str(), msg.c_str());
 
                        c->WriteChannelWithServ(ServerInstance->Config->ServerName, "PRIVMSG %s :%s", c->name, buf);
                        ServerInstance->PI->SendChannelPrivmsg(c, 0, buf);
+                       Logging = false;
                }
        }
 };
@@ -51,6 +56,10 @@ class ModuleChanLog : public Module
  public:
        ModuleChanLog(InspIRCd* Me) : Module(Me)
        {
+               Implementation eventlist[] = { I_OnRehash };
+               ServerInstance->Modules->Attach(eventlist, this, 1);
+
+               OnRehash(NULL, "");
        }
 
        virtual ~ModuleChanLog()
@@ -63,23 +72,25 @@ class ModuleChanLog : public Module
                }
        }
 
-       virtual void OnReadConfig(ServerConfig* sc, ConfigReader* Conf)
+       virtual void OnRehash(User *user, const std::string &parameter)
        {
+               ConfigReader Conf(ServerInstance);
+
                /* Since the CloseLogs prior to this hook just wiped out our logstreams for us, we just need to wipe the vector. */
                std::vector<ChannelLogStream*>().swap(cls);
-               int index, max = Conf->Enumerate("log");
+               int index, max = Conf.Enumerate("log");
                cls.reserve(max);
 
                for (index = 0; index < max; ++index)
                {
-                       std::string method = Conf->ReadValue("log", "method", index);
+                       std::string method = Conf.ReadValue("log", "method", index);
 
 
                        //if (method != "file")
                        //      continue;
 
-                       std::string type = Conf->ReadValue("log", "type", index);
-                       std::string level = Conf->ReadValue("log", "level", index);
+                       std::string type = Conf.ReadValue("log", "type", index);
+                       std::string level = Conf.ReadValue("log", "level", index);
                        int loglevel = DEFAULT;
 
                        if (level == "debug")
@@ -104,10 +115,7 @@ class ModuleChanLog : public Module
                                loglevel = NONE;
                        }
 
-                       std::string target = Conf->ReadValue("log", "target", index);
-
-printf("looking at tag with method: %s type: %s level: %s target: %s", method.c_str(), type.c_str(), level.c_str(), target.c_str());
-
+                       std::string target = Conf.ReadValue("log", "target", index);
                        ChannelLogStream* c = new ChannelLogStream(ServerInstance, loglevel, target);
                        ServerInstance->Logs->AddLogTypes(type, c, true);
                        cls.push_back(c);