]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/configreader.cpp
add a hint to users for using an IP instead of *
[user/henk/code/inspircd.git] / src / configreader.cpp
index 2b6672ae9d33f425c6567ab986cefeec9f642666..42294412b93304a887dc7169811bb228edecf7bf 100644 (file)
@@ -53,7 +53,6 @@ ServerConfig::ServerConfig(InspIRCd* Instance) : ServerInstance(Instance)
        debugging = 0;
        MaxChans = 20;
        OperMaxChans = 30;
-       LogLevel = DEFAULT;
        maxbans.clear();
        DNSServerValidator = &ValidateDnsServer;
 }
@@ -129,9 +128,9 @@ void ServerConfig::Send005(User* user)
                user->WriteNumeric(005, "%s %s", user->nick, line->c_str());
 }
 
-bool ServerConfig::CheckOnce(const char* tag)
+bool ServerConfig::CheckOnce(const char* tag, ConfigDataHash &newconf)
 {
-       int count = ConfValueEnum(this->config_data, tag);
+       int count = ConfValueEnum(newconf, tag);
 
        if (count > 1)
        {
@@ -368,27 +367,6 @@ bool ValidateMaxWho(ServerConfig* conf, const char*, const char*, ValueItem &dat
        return true;
 }
 
-bool ValidateLogLevel(ServerConfig* conf, const char*, const char*, ValueItem &data)
-{
-       std::string dbg = data.GetString();
-       conf->LogLevel = DEFAULT;
-
-       if (dbg == "debug")
-               conf->LogLevel = DEBUG;
-       else if (dbg  == "verbose")
-               conf->LogLevel = VERBOSE;
-       else if (dbg == "default")
-               conf->LogLevel = DEFAULT;
-       else if (dbg == "sparse")
-               conf->LogLevel = SPARSE;
-       else if (dbg == "none")
-               conf->LogLevel = NONE;
-
-       conf->debugging = (conf->LogLevel == DEBUG);
-
-       return true;
-}
-
 bool ValidateMotd(ServerConfig* conf, const char*, const char*, ValueItem &data)
 {
        conf->ReadFile(conf->MOTD, data.GetString());
@@ -581,6 +559,15 @@ bool DoConnect(ServerConfig* conf, const char*, char**, ValueList &values, int*)
        {
                if (*allow)
                {
+                       /* Find existing class by mask, the mask should be unique */
+                       for (ClassVector::iterator item = conf->Classes.begin(); item != conf->Classes.end(); ++item)
+                       {
+                               if ((*item)->GetHost() == allow)
+                               {
+                                       (*item)->Update(timeout, flood, allow, pingfreq, password, threshold, sendq, recvq, localmax, globalmax, maxchans, port, limit);
+                                       return true;
+                               }
+                       }
                        ConnectClass* cc = new ConnectClass(name, timeout, flood, allow, pingfreq, password, hashtype, threshold, sendq, recvq, localmax, globalmax, maxchans);
                        cc->limit = limit;
                        cc->SetPort(port);
@@ -588,6 +575,16 @@ bool DoConnect(ServerConfig* conf, const char*, char**, ValueList &values, int*)
                }
                else
                {
+                       /* Find existing class by mask, the mask should be unique */
+                       for (ClassVector::iterator item = conf->Classes.begin(); item != conf->Classes.end(); ++item)
+                       {
+                               if ((*item)->GetHost() == deny)
+                               {
+                                       (*item)->Update(name, deny);
+                                       (*item)->SetPort(port);
+                                       return true;
+                               }
+                       }
                        ConnectClass* cc = new ConnectClass(name, deny);
                        cc->SetPort(port);
                        conf->Classes.push_back(cc);
@@ -753,7 +750,6 @@ void ServerConfig::Read(bool bail, User* user)
 {
        int rem = 0, add = 0;      /* Number of modules added, number of modules removed */
 
-       static char debug[MAXBUF];      /* Temporary buffer for debugging value */
        static char maxkeep[MAXBUF];    /* Temporary buffer for WhoWasMaxKeep value */
        static char hidemodes[MAXBUF];  /* Modes to not allow listing from users below halfop */
        static char exemptchanops[MAXBUF];      /* Exempt channel ops from these modes */
@@ -789,7 +785,6 @@ void ServerConfig::Read(bool bail, User* user)
                {"options",     "prefixpart",   "",                     new ValueContainerChar (this->PrefixPart),              DT_CHARPTR,  NoValidation},
                {"options",     "suffixpart",   "",                     new ValueContainerChar (this->SuffixPart),              DT_CHARPTR,  NoValidation},
                {"options",     "fixedpart",    "",                     new ValueContainerChar (this->FixedPart),               DT_CHARPTR,  NoValidation},
-               {"options",     "loglevel",     "default",              new ValueContainerChar (debug),                         DT_CHARPTR,  ValidateLogLevel},
                {"options",     "netbuffersize","10240",                new ValueContainerInt  (&this->NetBufferSize),          DT_INTEGER,  ValidateNetBufferSize},
                {"options",     "maxwho",       "128",                  new ValueContainerInt  (&this->MaxWhoResults),          DT_INTEGER,  ValidateMaxWho},
                {"options",     "allowhalfop",  "0",                    new ValueContainerBool (&this->AllowHalfop),            DT_BOOLEAN,  NoValidation},
@@ -911,14 +906,7 @@ void ServerConfig::Read(bool bail, User* user)
        /* Make a copy here so if it fails then we can carry on running with an unaffected config */
        newconfig.clear();
 
-       if (this->DoInclude(newconfig, ServerInstance->ConfigFileName, errstr))
-       {
-               /* If we succeeded, set the ircd config to the new one */
-               ServerInstance->Threads->Mutex(true);
-               this->config_data = newconfig;
-               ServerInstance->Threads->Mutex(false);
-       }
-       else
+       if (!this->DoInclude(newconfig, ServerInstance->ConfigFileName, errstr))
        {
                ReportConfigError(errstr.str(), bail, user);
                return;
@@ -930,7 +918,7 @@ void ServerConfig::Read(bool bail, User* user)
                /* Check we dont have more than one of singular tags, or any of them missing
                 */
                for (int Index = 0; Once[Index]; Index++)
-                       if (!CheckOnce(Once[Index]))
+                       if (!CheckOnce(Once[Index], newconfig))
                                return;
 
                /* Read the values of all the tags which occur once or not at all, and call their callbacks.
@@ -950,7 +938,7 @@ void ServerConfig::Read(bool bail, User* user)
                        if (bootonly && !bail)
                                continue;
 
-                       ConfValue(this->config_data, Values[Index].tag, Values[Index].value, Values[Index].default_value, 0, item, MAXBUF, allow_newlines);
+                       ConfValue(newconfig, Values[Index].tag, Values[Index].value, Values[Index].default_value, 0, item, MAXBUF, allow_newlines);
                        ValueItem vi(item);
                        
                        if (!Values[Index].validation_function(this, Values[Index].tag, Values[Index].value, vi))
@@ -1032,7 +1020,7 @@ void ServerConfig::Read(bool bail, User* user)
                        MultiValues[Index].init_function(this, MultiValues[Index].tag);
                        ServerInstance->Threads->Mutex(false);
 
-                       int number_of_tags = ConfValueEnum(this->config_data, MultiValues[Index].tag);
+                       int number_of_tags = ConfValueEnum(newconfig, MultiValues[Index].tag);
 
                        for (int tagnum = 0; tagnum < number_of_tags; tagnum++)
                        {
@@ -1055,7 +1043,7 @@ void ServerConfig::Read(bool bail, User* user)
                                                        case DT_NOSPACES:
                                                        {
                                                                char item[MAXBUF];
-                                                               if (ConfValue(this->config_data, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
+                                                               if (ConfValue(newconfig, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
                                                                        vl.push_back(ValueItem(item));
                                                                else
                                                                        vl.push_back(ValueItem(""));
@@ -1065,7 +1053,7 @@ void ServerConfig::Read(bool bail, User* user)
                                                        case DT_HOSTNAME:
                                                        {
                                                                char item[MAXBUF];
-                                                               if (ConfValue(this->config_data, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
+                                                               if (ConfValue(newconfig, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
                                                                        vl.push_back(ValueItem(item));
                                                                else
                                                                        vl.push_back(ValueItem(""));
@@ -1075,7 +1063,7 @@ void ServerConfig::Read(bool bail, User* user)
                                                        case DT_IPADDRESS:
                                                        {
                                                                char item[MAXBUF];
-                                                               if (ConfValue(this->config_data, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
+                                                               if (ConfValue(newconfig, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
                                                                        vl.push_back(ValueItem(item));
                                                                else
                                                                        vl.push_back(ValueItem(""));
@@ -1085,7 +1073,7 @@ void ServerConfig::Read(bool bail, User* user)
                                                        case DT_CHANNEL:
                                                        {
                                                                char item[MAXBUF];
-                                                               if (ConfValue(this->config_data, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
+                                                               if (ConfValue(newconfig, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
                                                                        vl.push_back(ValueItem(item));
                                                                else
                                                                        vl.push_back(ValueItem(""));
@@ -1096,7 +1084,7 @@ void ServerConfig::Read(bool bail, User* user)
                                                        case DT_CHARPTR:
                                                        {
                                                                char item[MAXBUF];
-                                                               if (ConfValue(this->config_data, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
+                                                               if (ConfValue(newconfig, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item, MAXBUF, allow_newlines))
                                                                        vl.push_back(ValueItem(item));
                                                                else
                                                                        vl.push_back(ValueItem(""));
@@ -1105,7 +1093,7 @@ void ServerConfig::Read(bool bail, User* user)
                                                        case DT_INTEGER:
                                                        {
                                                                int item = 0;
-                                                               if (ConfValueInteger(this->config_data, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item))
+                                                               if (ConfValueInteger(newconfig, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum, item))
                                                                        vl.push_back(ValueItem(item));
                                                                else
                                                                        vl.push_back(ValueItem(0));
@@ -1113,7 +1101,7 @@ void ServerConfig::Read(bool bail, User* user)
                                                        break;
                                                        case DT_BOOLEAN:
                                                        {
-                                                               bool item = ConfValueBool(this->config_data, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum);
+                                                               bool item = ConfValueBool(newconfig, MultiValues[Index].tag, MultiValues[Index].items[valuenum], MultiValues[Index].items_default[valuenum], tagnum);
                                                                vl.push_back(ValueItem(item));
                                                        }
                                                        break;
@@ -1142,28 +1130,21 @@ void ServerConfig::Read(bool bail, User* user)
                return;
        }
 
-       // write once here, to try it out and make sure its ok
-       ServerInstance->WritePID(this->PID);
-
-       /* Switch over logfiles */
-       ServerInstance->Logs->CloseLogs();
-       ServerInstance->Logs->OpenFileLogs();
-
        ServerInstance->Threads->Mutex(true);
-       for (int i = 0; i < ConfValueEnum(this->config_data, "type"); ++i)
+       for (int i = 0; i < ConfValueEnum(newconfig, "type"); ++i)
        {
                char item[MAXBUF], classn[MAXBUF], classes[MAXBUF];
                std::string classname;
-               ConfValue(this->config_data, "type", "classes", "", i, classes, MAXBUF, false);
+               ConfValue(newconfig, "type", "classes", "", i, classes, MAXBUF, false);
                irc::spacesepstream str(classes);
-               ConfValue(this->config_data, "type", "name", "", i, item, MAXBUF, false);
+               ConfValue(newconfig, "type", "name", "", i, item, MAXBUF, false);
                while (str.GetToken(classname))
                {
                        std::string lost;
                        bool foundclass = false;
-                       for (int j = 0; j < ConfValueEnum(this->config_data, "class"); ++j)
+                       for (int j = 0; j < ConfValueEnum(newconfig, "class"); ++j)
                        {
-                               ConfValue(this->config_data, "class", "name", "", j, classn, MAXBUF, false);
+                               ConfValue(newconfig, "class", "name", "", j, classn, MAXBUF, false);
                                if (!strcmp(classn, classname.c_str()))
                                {
                                        foundclass = true;
@@ -1184,8 +1165,19 @@ void ServerConfig::Read(bool bail, User* user)
                        }
                }
        }
+
+       /* If we succeeded, set the ircd config to the new one */
+       this->config_data = newconfig;
+
        ServerInstance->Threads->Mutex(false);
 
+       // write once here, to try it out and make sure its ok
+       ServerInstance->WritePID(this->PID);
+
+       /* Switch over logfiles */
+       ServerInstance->Logs->CloseLogs();
+       ServerInstance->Logs->OpenFileLogs();
+
        ServerInstance->Logs->Log("CONFIG", DEFAULT, "Done reading configuration file.");
 
        /* If we're rehashing, let's load any new modules, and unload old ones