]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/configreader.cpp
Merge pull request #1162 from SaberUK/insp20+fix-deinstall
[user/henk/code/inspircd.git] / src / configreader.cpp
index 20e244dcc74c6b4f71de35de1d665768e6a8786c..5b298ddd8066b6ae00e2ac7f3fd124504ea3fede 100644 (file)
-/*       +------------------------------------+
- *       | Inspire Internet Relay Chat Daemon |
- *       +------------------------------------+
+/*
+ * InspIRCd -- Internet Relay Chat Daemon
  *
- *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
- * See: http://wiki.inspircd.org/Credits
+ *   Copyright (C) 2009-2010 Daniel De Graaf <danieldg@inspircd.org>
+ *   Copyright (C) 2007-2009 Robin Burchell <robin+git@viroteck.net>
+ *   Copyright (C) 2006-2009 Dennis Friis <peavey@inspircd.org>
+ *   Copyright (C) 2006-2008 Craig Edwards <craigedwards@brainbox.cc>
+ *   Copyright (C) 2008 Thomas Stagner <aquanight@inspircd.org>
+ *   Copyright (C) 2006 Oliver Lupton <oliverlupton@gmail.com>
  *
- * This program is free but copyrighted software; see
- *         the file COPYING for details.
+ * This file is part of InspIRCd.  InspIRCd is free software: you can
+ * redistribute it and/or modify it under the terms of the GNU General Public
+ * License as published by the Free Software Foundation, version 2.
  *
- * ---------------------------------------------------
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/* $Core */
-/* $CopyInstall: conf/inspircd.quotes.example $(CONPATH) */
-/* $CopyInstall: conf/inspircd.rules.example $(CONPATH) */
-/* $CopyInstall: conf/inspircd.motd.example $(CONPATH) */
-/* $CopyInstall: conf/inspircd.helpop-full.example $(CONPATH) */
-/* $CopyInstall: conf/inspircd.helpop.example $(CONPATH) */
-/* $CopyInstall: conf/inspircd.censor.example $(CONPATH) */
-/* $CopyInstall: conf/inspircd.filter.example $(CONPATH) */
-/* $CopyInstall: conf/inspircd.conf.example $(CONPATH) */
-/* $CopyInstall: conf/modules.conf.example $(CONPATH) */
-/* $CopyInstall: conf/opers.conf.example $(CONPATH) */
-/* $CopyInstall: conf/links.conf.example $(CONPATH) */
-/* $CopyInstall: tools/gdbargs $(BASE)/.gdbargs */
 
 #include "inspircd.h"
 #include <fstream>
 #include "xline.h"
 #include "exitcodes.h"
 #include "commands/cmd_whowas.h"
-#include "modes/cmode_h.h"
-
-struct fpos
-{
-       std::string filename;
-       int line;
-       int col;
-       fpos(const std::string& name, int l = 1, int c = 1) : filename(name), line(l), col(c) {}
-       std::string str()
-       {
-               return filename + ":" + ConvToStr(line) + ":" + ConvToStr(col);
-       }
-};
-
-enum ParseFlags
-{
-       FLAG_NO_EXEC = 1,
-       FLAG_NO_INC = 2
-};
-
-struct ParseStack
-{
-       std::vector<std::string> reading;
-       std::map<std::string, std::string> vars;
-       ConfigDataHash& output;
-       std::stringstream& errstr;
-
-       ParseStack(ServerConfig* conf)
-               : output(conf->config_data), errstr(conf->errstr)
-       {
-               vars["amp"] = "&";
-               vars["quot"] = "\"";
-               vars["newline"] = vars["nl"] = "\n";
-       }
-       bool ParseFile(const std::string& name, int flags);
-       bool ParseExec(const std::string& name, int flags);
-       void DoInclude(ConfigTag* includeTag, int flags);
-};
-
-struct Parser
-{
-       ParseStack& stack;
-       const int flags;
-       FILE* const file;
-       fpos current;
-       fpos last_tag;
-       reference<ConfigTag> tag;
-       int ungot;
-
-       Parser(ParseStack& me, int myflags, FILE* conf, const std::string& name)
-               : stack(me), flags(myflags), file(conf), current(name), last_tag(name), ungot(-1)
-       { }
-
-       int next(bool eof_ok = false)
-       {
-               if (ungot != -1)
-               {
-                       int ch = ungot;
-                       ungot = -1;
-                       return ch;
-               }
-               int ch = fgetc(file);
-               if (ch == EOF && !eof_ok)
-               {
-                       throw CoreException("Unexpected end-of-file");
-               }
-               else if (ch == '\n')
-               {
-                       current.line++;
-                       current.col = 0;
-               }
-               else
-               {
-                       current.col++;
-               }
-               return ch;
-       }
-
-       void unget(int ch)
-       {
-               if (ungot != -1)
-                       throw CoreException("INTERNAL ERROR: cannot unget twice");
-               ungot = ch;
-       }
-
-       void comment()
-       {
-               while (1)
-               {
-                       int ch = next();
-                       if (ch == '\n')
-                               return;
-               }
-       }
-
-       void nextword(std::string& rv)
-       {
-               int ch = next();
-               while (isspace(ch))
-                       ch = next();
-               while (isalnum(ch) || ch == '_')
-               {
-                       rv.push_back(ch);
-                       ch = next();
-               }
-               unget(ch);
-       }
-
-       bool kv()
-       {
-               std::string key;
-               nextword(key);
-               int ch = next();
-               if (ch == '>' && key.empty())
-               {
-                       return false;
-               }
-               else if (ch == '#' && key.empty())
-               {
-                       comment();
-                       return true;
-               }
-               else if (ch != '=')
-               {
-                       throw CoreException("Invalid character " + std::string(1, ch) + " in key (" + key + ")");
-               }
-
-               std::string value;
-               ch = next();
-               if (ch != '"')
-               {
-                       throw CoreException("Invalid character in value of <" + tag->tag + ":" + key + ">");
-               }
-               while (1)
-               {
-                       ch = next();
-                       if (ch == '&')
-                       {
-                               std::string varname;
-                               while (1)
-                               {
-                                       ch = next();
-                                       if (isalnum(ch))
-                                               varname.push_back(ch);
-                                       else if (ch == ';')
-                                               break;
-                                       else
-                                       {
-                                               stack.errstr << "Invalid XML entity name in value of <" + tag->tag + ":" + key + ">\n"
-                                                       << "To include an ampersand or quote, use &amp; or &quot;\n";
-                                               throw CoreException("Parse error");
-                                       }
-                               }
-                               std::map<std::string, std::string>::iterator var = stack.vars.find(varname);
-                               if (var == stack.vars.end())
-                                       throw CoreException("Undefined XML entity reference '&" + varname + ";'");
-                               value.append(var->second);
-                       }
-                       else if (ch == '"')
-                               break;
-                       value.push_back(ch);
-               }
-               tag->items.push_back(KeyVal(key, value));
-               return true;
-       }
-
-       void dotag()
-       {
-               last_tag = current;
-               std::string name;
-               nextword(name);
-
-               int spc = next();
-               if (spc == '>')
-                       unget(spc);
-               else if (!isspace(spc))
-                       throw CoreException("Invalid character in tag name");
-
-               if (name.empty())
-                       throw CoreException("Empty tag name");
-
-               tag = new ConfigTag(name, current.filename, current.line);
-
-               while (kv());
-
-               if (name == "include")
-               {
-                       stack.DoInclude(tag, flags);
-               }
-               else if (name == "define")
-               {
-                       std::string varname = tag->getString("name");
-                       std::string value = tag->getString("value");
-                       if (varname.empty())
-                               throw CoreException("Variable definition must include variable name");
-                       stack.vars[varname] = value;
-               }
-               else
-               {
-                       stack.output.insert(std::make_pair(name, tag));
-               }
-               // this is not a leak; reference<> takes care of the delete
-               tag = NULL;
-       }
-
-       bool outer_parse()
-       {
-               try
-               {
-                       while (1)
-                       {
-                               int ch = next(true);
-                               switch (ch)
-                               {
-                                       case EOF:
-                                               // this is the one place where an EOF is not an error
-                                               return true;
-                                       case '#':
-                                               comment();
-                                               break;
-                                       case '<':
-                                               dotag();
-                                               break;
-                                       case ' ':
-                                       case '\r':
-                                       case '\t':
-                                       case '\n':
-                                               break;
-                                       case 0xFE:
-                                       case 0xFF:
-                                               stack.errstr << "Do not save your files as UTF-16; use ASCII!\n";
-                                       default:
-                                               throw CoreException("Syntax error - start of tag expected");
-                               }
-                       }
-               }
-               catch (CoreException& err)
-               {
-                       stack.errstr << err.GetReason() << " at " << current.str();
-                       if (tag)
-                               stack.errstr << " (inside tag " << tag->tag << " at line " << tag->src_line << ")\n";
-                       else
-                               stack.errstr << " (last tag was on line " << last_tag.line << ")\n";
-               }
-               return false;
-       }
-};
-
-void ParseStack::DoInclude(ConfigTag* tag, int flags)
-{
-       if (flags & FLAG_NO_INC)
-               throw CoreException("Invalid <include> tag in file included with noinclude=\"yes\"");
-       std::string name;
-       if (tag->readString("file", name))
-       {
-               if (tag->getBool("noinclude", false))
-                       flags |= FLAG_NO_INC;
-               if (tag->getBool("noexec", false))
-                       flags |= FLAG_NO_EXEC;
-               if (!ParseFile(name, flags))
-                       throw CoreException("Included");
-       }
-       else if (tag->readString("executable", name))
-       {
-               if (flags & FLAG_NO_EXEC)
-                       throw CoreException("Invalid <include:executable> tag in file included with noexec=\"yes\"");
-               if (tag->getBool("noinclude", false))
-                       flags |= FLAG_NO_INC;
-               if (tag->getBool("noexec", true))
-                       flags |= FLAG_NO_EXEC;
-               if (!ParseExec(name, flags))
-                       throw CoreException("Included");
-       }
-}
-
-/** RAII wrapper on FILE* to close files on exceptions */
-struct FileWrapper
-{
-       FILE* const f;
-       FileWrapper(FILE* file) : f(file) {}
-       operator bool() { return f; }
-       operator FILE*() { return f; }
-       ~FileWrapper()
-       {
-               if (f)
-                       fclose(f);
-       }
-};
-
-bool ParseStack::ParseFile(const std::string& name, int flags)
-{
-       ServerInstance->Logs->Log("CONFIG", DEBUG, "Reading file %s", name.c_str());
-       for (unsigned int t = 0; t < reading.size(); t++)
-       {
-               if (std::string(name) == reading[t])
-               {
-                       throw CoreException("File " + name + " is included recursively (looped inclusion)");
-               }
-       }
-
-       /* It's not already included, add it to the list of files we've loaded */
-
-       FileWrapper file(fopen(name.c_str(), "r"));
-       if (!file)
-               throw CoreException("Could not read \"" + name + "\" for include");
-
-       reading.push_back(name);
-       Parser p(*this, flags, file, name);
-       bool ok = p.outer_parse();
-       reading.pop_back();
-       return ok;
-}
-
-bool ParseStack::ParseExec(const std::string& name, int flags)
-{
-       ServerInstance->Logs->Log("CONFIG", DEBUG, "Reading executable %s", name.c_str());
-       for (unsigned int t = 0; t < reading.size(); t++)
-       {
-               if (std::string(name) == reading[t])
-               {
-                       throw CoreException("Executable " + name + " is included recursively (looped inclusion)");
-               }
-       }
-
-       /* It's not already included, add it to the list of files we've loaded */
-
-       FileWrapper file(popen(name.c_str(), "r"));
-       if (!file)
-               throw CoreException("Could not open executable \"" + name + "\" for include");
-
-       reading.push_back(name);
-       Parser p(*this, flags, file, name);
-       bool ok = p.outer_parse();
-       reading.pop_back();
-       return ok;
-}
-
-/////////////////////////////////////////////////////////////////////////////
+#include "configparser.h"
+#include <iostream>
+#ifdef _WIN32
+#include <Iphlpapi.h>
+#pragma comment(lib, "Iphlpapi.lib")
+#endif
 
 ServerConfig::ServerConfig()
+       : NoSnoticeStack(false)
 {
        WhoWasGroupSize = WhoWasMaxGroups = WhoWasMaxKeep = 0;
-       log_file = NULL;
-       NoUserDns = forcedebug = OperSpyWhois = nofork = HideBans = HideSplits = UndernetMsgPrefix = false;
-       CycleHosts = writelog = AllowHalfop = InvBypassModes = true;
-       dns_timeout = DieDelay = 5;
+       RawLog = NoUserDns = HideBans = HideSplits = UndernetMsgPrefix = false;
+       WildcardIPv6 = CycleHosts = InvBypassModes = true;
+       dns_timeout = 5;
        MaxTargets = 20;
        NetBufferSize = 10240;
        SoftLimit = ServerInstance->SE->GetMaxFds();
        MaxConn = SOMAXCONN;
-       MaxWhoResults = 0;
-       debugging = 0;
        MaxChans = 20;
        OperMaxChans = 30;
        c_ipv4_range = 32;
        c_ipv6_range = 128;
+
+       std::vector<KeyVal>* items;
+       EmptyTag = ConfigTag::create("empty", "<auto>", 0, items);
+}
+
+ServerConfig::~ServerConfig()
+{
+       delete EmptyTag;
 }
 
 void ServerConfig::Update005()
 {
        std::stringstream out(data005);
+       std::vector<std::string> data;
        std::string token;
+       while (out >> token)
+               data.push_back(token);
+       sort(data.begin(), data.end());
+
        std::string line5;
-       int token_counter = 0;
        isupport.clear();
-       while (out >> token)
+       for(unsigned int i=0; i < data.size(); i++)
        {
+               token = data[i];
                line5 = line5 + token + " ";
-               token_counter++;
-               if (token_counter >= 13)
+               if (i % 13 == 12)
                {
-                       char buf[MAXBUF];
-                       snprintf(buf, MAXBUF, "%s:are supported by this server", line5.c_str());
-                       isupport.push_back(buf);
+                       line5.append(":are supported by this server");
+                       isupport.push_back(line5);
                        line5.clear();
-                       token_counter = 0;
                }
        }
        if (!line5.empty())
        {
-               char buf[MAXBUF];
-               snprintf(buf, MAXBUF, "%s:are supported by this server", line5.c_str());
-               isupport.push_back(buf);
+               line5.append(":are supported by this server");
+               isupport.push_back(line5);
        }
 }
 
@@ -422,13 +94,6 @@ void ServerConfig::Send005(User* user)
                user->WriteNumeric(RPL_ISUPPORT, "%s %s", user->nick.c_str(), line->c_str());
 }
 
-static void ReqRead(ServerConfig* src, const std::string& tag, const std::string& key, std::string& dest)
-{
-       ConfigTag* t = src->ConfValue(tag);
-       if (!t || !t->readString(key, dest))
-               throw CoreException("You must specify a value for <" + tag + ":" + key + ">");
-}
-
 template<typename T, typename V>
 static void range(T& value, V min, V max, V def, const char* msg)
 {
@@ -441,63 +106,11 @@ static void range(T& value, V min, V max, V def, const char* msg)
 }
 
 
-/* NOTE: Before anyone asks why we're not using inet_pton for this, it is because inet_pton and friends do not return so much detail,
- * even in strerror(errno). They just return 'yes' or 'no' to an address without such detail as to whats WRONG with the address.
- * Because ircd users arent as technical as they used to be (;)) we are going to give more of a useful error message.
- */
 static void ValidIP(const std::string& ip, const std::string& key)
 {
-       const char* p = ip.c_str();
-       int num_dots = 0;
-       int num_seps = 0;
-       int not_numbers = false;
-       int not_hex = false;
-
-       if (*p)
-       {
-               if (*p == '.')
-                       throw CoreException("The value of "+key+" is not an IP address");
-
-               for (const char* ptr = p; *ptr; ++ptr)
-               {
-                       if (*ptr != ':' && *ptr != '.')
-                       {
-                               if (*ptr < '0' || *ptr > '9')
-                                       not_numbers = true;
-                               if ((*ptr < '0' || *ptr > '9') && (toupper(*ptr) < 'A' || toupper(*ptr) > 'F'))
-                                       not_hex = true;
-                       }
-                       switch (*ptr)
-                       {
-                               case ' ':
-                                       throw CoreException("The value of "+key+" is not an IP address");
-                               case '.':
-                                       num_dots++;
-                               break;
-                               case ':':
-                                       num_seps++;
-                               break;
-                       }
-               }
-
-               if (num_dots > 3)
-                       throw CoreException("The value of "+key+" is an IPv4 address with too many fields!");
-
-               if (num_seps > 8)
-                       throw CoreException("The value of "+key+" is an IPv6 address with too many fields!");
-
-               if (num_seps == 0 && num_dots < 3)
-                       throw CoreException("The value of "+key+" looks to be a malformed IPv4 address");
-
-               if (num_seps == 0 && num_dots == 3 && not_numbers)
-                       throw CoreException("The value of "+key+" contains non-numeric characters in an IPv4 address");
-
-               if (num_seps != 0 && not_hex)
-                       throw CoreException("The value of "+key+" contains non-hexdecimal characters in an IPv6 address");
-
-               if (num_seps != 0 && num_dots != 3 && num_dots != 0)
-                       throw CoreException("The value of "+key+" is a malformed IPv6 4in6 address");
-       }
+       irc::sockets::sockaddrs dummy;
+       if (!irc::sockets::aptosa(ip, 0, dummy))
+               throw CoreException("The value of "+key+" is not an IP address");
 }
 
 static void ValidHost(const std::string& p, const std::string& msg)
@@ -520,8 +133,6 @@ static void ValidHost(const std::string& p, const std::string& msg)
                throw CoreException("The value of "+msg+" is not a valid hostname");
 }
 
-// Specialized validators
-
 bool ServerConfig::ApplyDisabledCommands(const std::string& data)
 {
        std::stringstream dcmds(data);
@@ -543,15 +154,41 @@ bool ServerConfig::ApplyDisabledCommands(const std::string& data)
        return true;
 }
 
-#ifdef WINDOWS
-// Note: the windows validator is in win32wrapper.cpp
-void FindDNS(std::string& server);
-#else
 static void FindDNS(std::string& server)
 {
        if (!server.empty())
                return;
+#ifdef _WIN32
+       // attempt to look up their nameserver from the system
+       ServerInstance->Logs->Log("CONFIG",DEFAULT,"WARNING: <dns:server> not defined, attempting to find a working server in the system settings...");
+
+       PFIXED_INFO pFixedInfo;
+       DWORD dwBufferSize = sizeof(FIXED_INFO);
+       pFixedInfo = (PFIXED_INFO) HeapAlloc(GetProcessHeap(), 0, sizeof(FIXED_INFO));
 
+       if(pFixedInfo)
+       {
+               if (GetNetworkParams(pFixedInfo, &dwBufferSize) == ERROR_BUFFER_OVERFLOW) {
+                       HeapFree(GetProcessHeap(), 0, pFixedInfo);
+                       pFixedInfo = (PFIXED_INFO) HeapAlloc(GetProcessHeap(), 0, dwBufferSize);
+               }
+
+               if(pFixedInfo) {
+                       if (GetNetworkParams(pFixedInfo, &dwBufferSize) == NO_ERROR)
+                               server = pFixedInfo->DnsServerList.IpAddress.String;
+
+                       HeapFree(GetProcessHeap(), 0, pFixedInfo);
+               }
+
+               if(!server.empty())
+               {
+                       ServerInstance->Logs->Log("CONFIG",DEFAULT,"<dns:server> set to '%s' as first active resolver in the system settings.", server.c_str());
+                       return;
+               }
+       }
+
+       ServerInstance->Logs->Log("CONFIG",DEFAULT,"No viable nameserver found! Defaulting to nameserver '127.0.0.1'!");
+#else
        // attempt to look up their nameserver from /etc/resolv.conf
        ServerInstance->Logs->Log("CONFIG",DEFAULT,"WARNING: <dns:server> not defined, attempting to find working server in /etc/resolv.conf...");
 
@@ -562,15 +199,18 @@ static void FindDNS(std::string& server)
                if (server == "nameserver")
                {
                        resolv >> server;
-                       ServerInstance->Logs->Log("CONFIG",DEFAULT,"<dns:server> set to '%s' as first resolver in /etc/resolv.conf.",server.c_str());
-                       return;
+                       if (server.find_first_not_of("0123456789.") == std::string::npos)
+                       {
+                               ServerInstance->Logs->Log("CONFIG",DEFAULT,"<dns:server> set to '%s' as first resolver in /etc/resolv.conf.",server.c_str());
+                               return;
+                       }
                }
        }
 
        ServerInstance->Logs->Log("CONFIG",DEFAULT,"/etc/resolv.conf contains no viable nameserver entries! Defaulting to nameserver '127.0.0.1'!");
+#endif
        server = "127.0.0.1";
 }
-#endif
 
 static void ReadXLine(ServerConfig* conf, const std::string& tag, const std::string& key, XLineFactory* make)
 {
@@ -599,6 +239,8 @@ void ServerConfig::CrossCheckOperClassType()
                std::string name = tag->getString("name");
                if (name.empty())
                        throw CoreException("<class:name> missing from tag at " + tag->getTagLocation());
+               if (operclass.find(name) != operclass.end())
+                       throw CoreException("Duplicate class block with name " + name + " at " + tag->getTagLocation());
                operclass[name] = tag;
        }
        tags = ConfTags("type");
@@ -608,12 +250,14 @@ void ServerConfig::CrossCheckOperClassType()
                std::string name = tag->getString("name");
                if (name.empty())
                        throw CoreException("<type:name> is missing from tag at " + tag->getTagLocation());
-
                if (!ServerInstance->IsNick(name.c_str(), Limits.NickMax))
                        throw CoreException("<type:name> is invalid (value '" + name + "')");
+               if (oper_blocks.find(" " + name) != oper_blocks.end())
+                       throw CoreException("Duplicate type block with name " + name + " at " + tag->getTagLocation());
 
                OperInfo* ifo = new OperInfo;
                oper_blocks[" " + name] = ifo;
+               ifo->name = name;
                ifo->type_block = tag;
 
                std::string classname;
@@ -641,7 +285,7 @@ void ServerConfig::CrossCheckOperClassType()
                if (tblk == oper_blocks.end())
                        throw CoreException("Oper block " + name + " has missing type " + type);
                if (oper_blocks.find(name) != oper_blocks.end())
-                       throw CoreException("Duplicate oper block with name " + name);
+                       throw CoreException("Duplicate oper block with name " + name + " at " + tag->getTagLocation());
 
                OperInfo* ifo = new OperInfo;
                ifo->name = type;
@@ -661,21 +305,38 @@ void ServerConfig::CrossCheckConnectBlocks(ServerConfig* current)
                for(ClassVector::iterator i = current->Classes.begin(); i != current->Classes.end(); ++i)
                {
                        ConnectClass* c = *i;
-                       std::string typeMask = (c->type == CC_ALLOW) ? "a" : "d";
-                       typeMask += c->host;
-                       oldBlocksByMask[typeMask] = c;
+                       if (c->name.substr(0, 8) != "unnamed-")
+                       {
+                               oldBlocksByMask["n" + c->name] = c;
+                       }
+                       else if (c->type == CC_ALLOW || c->type == CC_DENY)
+                       {
+                               std::string typeMask = (c->type == CC_ALLOW) ? "a" : "d";
+                               typeMask += c->host;
+                               oldBlocksByMask[typeMask] = c;
+                       }
                }
        }
 
-       ClassMap newBlocksByMask;
-       Classes.resize(config_data.count("type"));
+       int blk_count = config_data.count("connect");
+       if (blk_count == 0)
+       {
+               // No connect blocks found; make a trivial default block
+               std::vector<KeyVal>* items;
+               ConfigTag* tag = ConfigTag::create("connect", "<auto>", 0, items);
+               items->push_back(std::make_pair("allow", "*"));
+               config_data.insert(std::make_pair("connect", tag));
+               blk_count = 1;
+       }
+
+       Classes.resize(blk_count);
        std::map<std::string, int> names;
 
        bool try_again = true;
        for(int tries=0; try_again; tries++)
        {
                try_again = false;
-               ConfigTagList tags = ConfTags("type");
+               ConfigTagList tags = ConfTags("connect");
                int i=0;
                for(ConfigIter it = tags.first; it != tags.second; ++it, ++i)
                {
@@ -692,21 +353,14 @@ void ServerConfig::CrossCheckConnectBlocks(ServerConfig* current)
                                {
                                        try_again = true;
                                        // couldn't find parent this time. If it's the last time, we'll never find it.
-                                       if (tries == 50)
-                                               throw CoreException("Could not find parent connect class \"" + parentName + "\" for connect block " + ConvToStr(i));
+                                       if (tries >= blk_count)
+                                               throw CoreException("Could not find parent connect class \"" + parentName + "\" for connect block at " + tag->getTagLocation());
                                        continue;
                                }
                                parent = Classes[parentIter->second];
                        }
 
                        std::string name = tag->getString("name");
-                       if (!name.empty())
-                       {
-                               if (names.find(name) != names.end())
-                                       throw CoreException("Two connect classes with name \"" + name + "\" defined!");
-                               names[name] = i;
-                       }
-
                        std::string mask, typeMask;
                        char type;
 
@@ -720,23 +374,36 @@ void ServerConfig::CrossCheckConnectBlocks(ServerConfig* current)
                                type = CC_DENY;
                                typeMask = 'd' + mask;
                        }
+                       else if (!name.empty())
+                       {
+                               type = CC_NAMED;
+                               mask = name;
+                               typeMask = 'n' + mask;
+                       }
+                       else
+                       {
+                               throw CoreException("Connect class must have allow, deny, or name specified at " + tag->getTagLocation());
+                       }
+
+                       if (name.empty())
+                       {
+                               name = "unnamed-" + ConvToStr(i);
+                       }
                        else
                        {
-                               throw CoreException("Connect class must have an allow or deny mask at " + tag->getTagLocation());
+                               typeMask = 'n' + name;
                        }
-                       ClassMap::iterator dupMask = newBlocksByMask.find(typeMask);
-                       if (dupMask != newBlocksByMask.end())
-                               throw CoreException("Two connect classes cannot have the same mask (" + mask + ")");
 
-                       ConnectClass* me = parent ? 
+                       if (names.find(name) != names.end())
+                               throw CoreException("Two connect classes with name \"" + name + "\" defined!");
+                       names[name] = i;
+
+                       ConnectClass* me = parent ?
                                new ConnectClass(tag, type, mask, *parent) :
                                new ConnectClass(tag, type, mask);
 
-                       if (!name.empty())
-                               me->name = name;
+                       me->name = name;
 
-                       tag->readString("password", me->pass);
-                       tag->readString("hash", me->hash);
                        me->registration_timeout = tag->getInt("timeout", me->registration_timeout);
                        me->pingtime = tag->getInt("pingfreq", me->pingtime);
                        std::string sendq;
@@ -754,10 +421,12 @@ void ServerConfig::CrossCheckConnectBlocks(ServerConfig* current)
                        me->hardsendqmax = tag->getInt("hardsendq", me->hardsendqmax);
                        me->recvqmax = tag->getInt("recvq", me->recvqmax);
                        me->penaltythreshold = tag->getInt("threshold", me->penaltythreshold);
+                       me->commandrate = tag->getInt("commandrate", me->commandrate);
+                       me->fakelag = tag->getBool("fakelag", me->fakelag);
                        me->maxlocal = tag->getInt("localmax", me->maxlocal);
                        me->maxglobal = tag->getInt("globalmax", me->maxglobal);
-                       me->port = tag->getInt("port", me->port);
                        me->maxchans = tag->getInt("maxchans", me->maxchans);
+                       me->maxconnwarn = tag->getBool("maxconnwarn", me->maxconnwarn);
                        me->limit = tag->getInt("limit", me->limit);
 
                        ClassMap::iterator oldMask = oldBlocksByMask.find(typeMask);
@@ -769,7 +438,6 @@ void ServerConfig::CrossCheckConnectBlocks(ServerConfig* current)
                                delete me;
                                me = old;
                        }
-                       newBlocksByMask[typeMask] = me;
                        Classes[i] = me;
                }
        }
@@ -808,20 +476,36 @@ static const Deprecated ChangedConfig[] = {
        {"options", "somaxconn",                "has been moved to <performance:somaxconn> as of 1.2a3"},
        {"options", "netbuffersize",    "has been moved to <performance:netbuffersize> as of 1.2a3"},
        {"options", "maxwho",           "has been moved to <performance:maxwho> as of 1.2a3"},
-       {"options",     "loglevel",             "1.2 does not use the loglevel value. Please define <log> tags instead."},
-       {"die",     "value",            "has always been deprecated"},
+       {"options",     "loglevel",             "1.2+ does not use the loglevel value. Please define <log> tags instead."},
+       {"die",     "value",            "you need to reread your config"},
+       {"bind",    "transport",                "has been moved to <bind:ssl> as of 2.0a1"},
+       {"link",    "transport",                "has been moved to <link:ssl> as of 2.0a1"},
+       {"link",        "autoconnect",          "2.0+ does not use the autoconnect value. Please define <autoconnect> tags instead."},
 };
 
 void ServerConfig::Fill()
 {
-       ReqRead(this, "server", "name", ServerName);
-       ReqRead(this, "power", "diepass", diepass);
-       ReqRead(this, "power", "restartpass", restartpass);
-
        ConfigTag* options = ConfValue("options");
        ConfigTag* security = ConfValue("security");
+       if (sid.empty())
+       {
+               ServerName = ConfValue("server")->getString("name");
+               sid = ConfValue("server")->getString("id");
+               ValidHost(ServerName, "<server:name>");
+               if (!sid.empty() && !ServerInstance->IsSID(sid))
+                       throw CoreException(sid + " is not a valid server ID. A server ID must be 3 characters long, with the first character a digit and the next two characters a digit or letter.");
+       }
+       else
+       {
+               if (ServerName != ConfValue("server")->getString("name"))
+                       throw CoreException("You must restart to change the server name or SID");
+               std::string nsid = ConfValue("server")->getString("id");
+               if (!nsid.empty() && nsid != sid)
+                       throw CoreException("You must restart to change the server name or SID");
+       }
+       diepass = ConfValue("power")->getString("diepass");
+       restartpass = ConfValue("power")->getString("restartpass");
        powerhash = ConfValue("power")->getString("hash");
-       DieDelay = ConfValue("power")->getInt("pause");
        PrefixQuit = options->getString("prefixquit");
        SuffixQuit = options->getString("suffixquit");
        FixedQuit = options->getString("fixedquit");
@@ -833,30 +517,27 @@ void ServerConfig::Fill()
        MoronBanner = options->getString("moronbanner", "You're banned!");
        ServerDesc = ConfValue("server")->getString("description", "Configure Me");
        Network = ConfValue("server")->getString("network", "Network");
-       sid = ConfValue("server")->getString("id", "");
        AdminName = ConfValue("admin")->getString("name", "");
        AdminEmail = ConfValue("admin")->getString("email", "null@example.com");
        AdminNick = ConfValue("admin")->getString("nick", "admin");
        ModPath = ConfValue("path")->getString("moduledir", MOD_PATH);
        NetBufferSize = ConfValue("performance")->getInt("netbuffersize", 10240);
-       MaxWhoResults = ConfValue("performance")->getInt("maxwho", 1024);
        dns_timeout = ConfValue("dns")->getInt("timeout", 5);
        DisabledCommands = ConfValue("disabled")->getString("commands", "");
        DisabledDontExist = ConfValue("disabled")->getBool("fakenonexistant");
-       SetUser = security->getString("runasuser");
-       SetGroup = security->getString("runasgroup");
        UserStats = security->getString("userstats");
-       CustomVersion = security->getString("customversion");
+       CustomVersion = security->getString("customversion", Network + " IRCd");
        HideSplits = security->getBool("hidesplits");
        HideBans = security->getBool("hidebans");
        HideWhoisServer = security->getString("hidewhois");
        HideKillsServer = security->getString("hidekills");
-       OperSpyWhois = security->getBool("operspywhois");
+       HideULineKills = security->getBool("hideulinekills");
        RestrictBannedUsers = security->getBool("restrictbannedusers", true);
        GenericOper = security->getBool("genericoper");
        NoUserDns = ConfValue("performance")->getBool("nouserdns");
        SyntaxHints = options->getBool("syntaxhints");
        CycleHosts = options->getBool("cyclehosts");
+       CycleHostsFromUser = options->getBool("cyclehostsfromuser");
        UndernetMsgPrefix = options->getBool("ircumsgprefix");
        FullHostInTopic = options->getBool("hostintopic");
        MaxTargets = security->getInt("maxtargets", 20);
@@ -865,11 +546,10 @@ void ServerConfig::Fill()
        WhoWasGroupSize = ConfValue("whowas")->getInt("groupsize");
        WhoWasMaxGroups = ConfValue("whowas")->getInt("maxgroups");
        WhoWasMaxKeep = ServerInstance->Duration(ConfValue("whowas")->getString("maxkeep"));
-       DieValue = ConfValue("die")->getString("value");
-       MaxChans = ConfValue("channels")->getInt("users");
-       OperMaxChans = ConfValue("channels")->getInt("opers");
-       c_ipv4_range = ConfValue("cidr")->getInt("ipv4clone");
-       c_ipv6_range = ConfValue("cidr")->getInt("ipv6clone");
+       MaxChans = ConfValue("channels")->getInt("users", 20);
+       OperMaxChans = ConfValue("channels")->getInt("opers", 60);
+       c_ipv4_range = ConfValue("cidr")->getInt("ipv4clone", 32);
+       c_ipv6_range = ConfValue("cidr")->getInt("ipv6clone", 128);
        Limits.NickMax = ConfValue("limits")->getInt("maxnick", 32);
        Limits.ChanMax = ConfValue("limits")->getInt("maxchan", 64);
        Limits.MaxModes = ConfValue("limits")->getInt("maxmodes", 20);
@@ -880,21 +560,38 @@ void ServerConfig::Fill()
        Limits.MaxGecos = ConfValue("limits")->getInt("maxgecos", 128);
        Limits.MaxAway = ConfValue("limits")->getInt("maxaway", 200);
        InvBypassModes = options->getBool("invitebypassmodes", true);
+       NoSnoticeStack = options->getBool("nosnoticestack", false);
+       WelcomeNotice = options->getBool("welcomenotice", true);
 
        range(SoftLimit, 10, ServerInstance->SE->GetMaxFds(), ServerInstance->SE->GetMaxFds(), "<performance:softlimit>");
-       range(MaxConn, 0, SOMAXCONN, SOMAXCONN, "<performance:somaxconn>");
+       if (ConfValue("performance")->getBool("limitsomaxconn", true))
+               range(MaxConn, 0, SOMAXCONN, SOMAXCONN, "<performance:somaxconn>");
        range(MaxTargets, 1, 31, 20, "<security:maxtargets>");
        range(NetBufferSize, 1024, 65534, 10240, "<performance:netbuffersize>");
-       range(MaxWhoResults, 1, 65535, 1024, "<performace:maxwho>");
        range(WhoWasGroupSize, 0, 10000, 10, "<whowas:groupsize>");
        range(WhoWasMaxGroups, 0, 1000000, 10240, "<whowas:maxgroups>");
        range(WhoWasMaxKeep, 3600, INT_MAX, 3600, "<whowas:maxkeep>");
 
        ValidIP(DNSServer, "<dns:server>");
-       ValidHost(ServerName, "<server:name>");
-       if (!sid.empty() && !ServerInstance->IsSID(sid))
-               throw CoreException(sid + " is not a valid server ID. A server ID must be 3 characters long, with the first character a digit and the next two characters a digit or letter.");
 
+       std::string defbind = options->getString("defaultbind");
+       if (assign(defbind) == "ipv4")
+       {
+               WildcardIPv6 = false;
+       }
+       else if (assign(defbind) == "ipv6")
+       {
+               WildcardIPv6 = true;
+       }
+       else
+       {
+               WildcardIPv6 = true;
+               int socktest = socket(AF_INET6, SOCK_STREAM, 0);
+               if (socktest < 0)
+                       WildcardIPv6 = false;
+               else
+                       ServerInstance->SE->Close(socktest);
+       }
        ConfigTagList tags = ConfTags("uline");
        for(ConfigIter i = tags.first; i != tags.second; ++i)
        {
@@ -921,22 +618,28 @@ void ServerConfig::Fill()
        ReadXLine(this, "exception", "host", ServerInstance->XLines->GetFactory("E"));
 
        memset(DisabledUModes, 0, sizeof(DisabledUModes));
-       for (const unsigned char* p = (const unsigned char*)ConfValue("disabled")->getString("usermodes").c_str(); *p; ++p)
+       std::string modes = ConfValue("disabled")->getString("usermodes");
+       for (std::string::const_iterator p = modes.begin(); p != modes.end(); ++p)
        {
-               if (*p < 'A' || *p > ('A' + 64)) throw CoreException(std::string("Invalid usermode ")+(char)*p+" was found.");
+               // Complain when the character is not a-z or A-Z
+               if ((*p < 'A') || (*p > 'z') || ((*p < 'a') && (*p > 'Z')))
+                       throw CoreException("Invalid usermode " + std::string(1, *p) + " was found.");
                DisabledUModes[*p - 'A'] = 1;
        }
 
        memset(DisabledCModes, 0, sizeof(DisabledCModes));
-       for (const unsigned char* p = (const unsigned char*)ConfValue("disabled")->getString("chanmodes").c_str(); *p; ++p)
+       modes = ConfValue("disabled")->getString("chanmodes");
+       for (std::string::const_iterator p = modes.begin(); p != modes.end(); ++p)
        {
-               if (*p < 'A' || *p > ('A' + 64)) throw CoreException(std::string("Invalid chanmode ")+(char)*p+" was found.");
+               if ((*p < 'A') || (*p > 'z') || ((*p < 'a') && (*p > 'Z')))
+                       throw CoreException("Invalid chanmode " + std::string(1, *p) + " was found.");
                DisabledCModes[*p - 'A'] = 1;
        }
 
        memset(HideModeLists, 0, sizeof(HideModeLists));
-       for (const unsigned char* p = (const unsigned char*)ConfValue("security")->getString("hidemodes").c_str(); *p; ++p)
-               HideModeLists[*p] = true;
+       modes = ConfValue("security")->getString("hidemodes");
+       for (std::string::const_iterator p = modes.begin(); p != modes.end(); ++p)
+               HideModeLists[(unsigned char) *p] = true;
 
        std::string v = security->getString("announceinvites");
 
@@ -949,12 +652,15 @@ void ServerConfig::Fill()
        else
                AnnounceInvites = ServerConfig::INVITE_ANNOUNCE_NONE;
 
-       Limits.Finalise();
+       v = security->getString("operspywhois");
+       if (v == "splitmsg")
+               OperSpyWhois = SPYWHOIS_SPLITMSG;
+       else if (v == "on" || v == "yes")
+               OperSpyWhois = SPYWHOIS_SINGLEMSG;
+       else
+               OperSpyWhois = SPYWHOIS_NONE;
 }
 
-/* These tags MUST occur and must ONLY occur once in the config file */
-static const char* const Once[] = { "server", "admin", "files", "power", "options" };
-
 // WARNING: it is not safe to use most of the codebase in this function, as it
 // will run in the config reader thread
 void ServerConfig::Read()
@@ -973,8 +679,6 @@ void ServerConfig::Read()
        }
        if (valid)
        {
-               ReadFile(MOTD, ConfValue("files")->getString("motd"));
-               ReadFile(RULES, ConfValue("files")->getString("rules"));
                DNSServer = ConfValue("dns")->getString("server");
                FindDNS(DNSServer);
        }
@@ -983,34 +687,30 @@ void ServerConfig::Read()
 void ServerConfig::Apply(ServerConfig* old, const std::string &useruid)
 {
        valid = true;
+       if (old)
+       {
+               /*
+                * These values can only be set on boot. Keep their old values. Do it before we send messages so we actually have a servername.
+                */
+               this->ServerName = old->ServerName;
+               this->sid = old->sid;
+               this->cmdline = old->cmdline;
+       }
 
        /* The stuff in here may throw CoreException, be sure we're in a position to catch it. */
        try
        {
-               /* Check we dont have more than one of singular tags, or any of them missing
-                */
-               for (int Index = 0; Index * sizeof(*Once) < sizeof(Once); Index++)
-               {
-                       std::string tag = Once[Index];
-                       if (!ConfValue(tag))
-                               throw CoreException("You have not defined a <"+tag+"> tag, this is required.");
-                       ConfigTagList tags = ConfTags(tag);
-                       if (tags.first != tags.second)
-                       {
-                               tags.first++;
-                               errstr << "You have more than one <" << tag << "> tag.\n"
-                                       << "First occurrence at " << ConfValue(tag)->getTagLocation()
-                                       << "; second occurrence at " << tags.first->second->getTagLocation() << std::endl;
-                       }
-               }
-
                for (int Index = 0; Index * sizeof(Deprecated) < sizeof(ChangedConfig); Index++)
                {
                        std::string dummy;
-                       if (ConfValue(ChangedConfig[Index].tag)->readString(ChangedConfig[Index].value, dummy, true))
-                               errstr << "Your configuration contains a deprecated value: <"
-                                       << ChangedConfig[Index].tag << ":" << ChangedConfig[Index].value << "> - " << ChangedConfig[Index].reason
-                                       << " (at " << ConfValue(ChangedConfig[Index].tag)->getTagLocation() << ")\n";
+                       ConfigTagList tags = ConfTags(ChangedConfig[Index].tag);
+                       for(ConfigIter i = tags.first; i != tags.second; ++i)
+                       {
+                               if (i->second->readString(ChangedConfig[Index].value, dummy, true))
+                                       errstr << "Your configuration contains a deprecated value: <"
+                                               << ChangedConfig[Index].tag << ":" << ChangedConfig[Index].value << "> - " << ChangedConfig[Index].reason
+                                               << " (at " << i->second->getTagLocation() << ")\n";
+                       }
                }
 
                Fill();
@@ -1024,20 +724,16 @@ void ServerConfig::Apply(ServerConfig* old, const std::string &useruid)
                errstr << ce.GetReason();
        }
 
+       // Check errors before dealing with failed binds, since continuing on failed bind is wanted in some circumstances.
+       valid = errstr.str().empty();
+
        // write once here, to try it out and make sure its ok
-       ServerInstance->WritePID(this->PID);
+       if (valid)
+               ServerInstance->WritePID(this->PID);
 
-       /*
-        * These values can only be set on boot. Keep their old values. Do it before we send messages so we actually have a servername.
-        */
-       if (old)
+       if (old && valid)
        {
-               this->ServerName = old->ServerName;
-               this->sid = old->sid;
-               this->argv = old->argv;
-               this->argc = old->argc;
-
-               // Same for ports... they're bound later on first run.
+               // On first run, ports are bound later on
                FailedPortList pl;
                ServerInstance->BindPorts(pl);
                if (pl.size())
@@ -1056,32 +752,48 @@ void ServerConfig::Apply(ServerConfig* old, const std::string &useruid)
 
        User* user = useruid.empty() ? NULL : ServerInstance->FindNick(useruid);
 
-       valid = errstr.str().empty();
        if (!valid)
+       {
                ServerInstance->Logs->Log("CONFIG",DEFAULT, "There were errors in your configuration file:");
+               Classes.clear();
+       }
 
        while (errstr.good())
        {
                std::string line;
                getline(errstr, line, '\n');
-               if (!line.empty())
-               {
-                       if (user)
-                               user->WriteServ("NOTICE %s :*** %s", user->nick.c_str(), line.c_str());
-                       else
-                               ServerInstance->SNO->WriteGlobalSno('a', line);
-               }
-
+               if (line.empty())
+                       continue;
+               // On startup, print out to console (still attached at this point)
                if (!old)
-               {
-                       // Starting up, so print it out so it's seen. XXX this is a bit of a hack.
-                       printf("%s\n", line.c_str());
-               }
+                       std::cout << line << std::endl;
+               // If a user is rehashing, tell them directly
+               if (user)
+                       user->SendText(":%s NOTICE %s :*** %s", ServerInstance->Config->ServerName.c_str(), user->nick.c_str(), line.c_str());
+               // Also tell opers
+               ServerInstance->SNO->WriteGlobalSno('a', line);
        }
 
        errstr.clear();
        errstr.str(std::string());
 
+       // Re-parse our MOTD and RULES files for colors -- Justasic
+       for (ClassVector::const_iterator it = this->Classes.begin(), it_end = this->Classes.end(); it != it_end; ++it)
+       {
+               ConfigTag *tag = (*it)->config;
+               // Make sure our connection class allows motd colors
+               if(!tag->getBool("allowmotdcolors"))
+                     continue;
+
+               ConfigFileCache::iterator file = this->Files.find(tag->getString("motd", "motd"));
+               if (file != this->Files.end())
+                     InspIRCd::ProcessColors(file->second);
+
+               file = this->Files.find(tag->getString("rules", "rules"));
+               if (file != this->Files.end())
+                     InspIRCd::ProcessColors(file->second);
+       }
+
        /* No old configuration -> initial boot, nothing more to do here */
        if (!old)
        {
@@ -1090,12 +802,10 @@ void ServerConfig::Apply(ServerConfig* old, const std::string &useruid)
                        ServerInstance->Exit(EXIT_STATUS_CONFIG);
                }
 
-               if (ConfValue("options")->getBool("allowhalfop"))
-                       ServerInstance->Modes->AddMode(new ModeChannelHalfOp);
-
                return;
        }
 
+
        // If there were errors processing configuration, don't touch modules.
        if (!valid)
                return;
@@ -1103,24 +813,13 @@ void ServerConfig::Apply(ServerConfig* old, const std::string &useruid)
        ApplyModules(user);
 
        if (user)
-               user->WriteServ("NOTICE %s :*** Successfully rehashed server.", user->nick.c_str());
+               user->SendText(":%s NOTICE %s :*** Successfully rehashed server.",
+                       ServerInstance->Config->ServerName.c_str(), user->nick.c_str());
        ServerInstance->SNO->WriteGlobalSno('a', "*** Successfully rehashed server.");
 }
 
 void ServerConfig::ApplyModules(User* user)
 {
-       bool AllowHalfOp = ConfValue("options")->getBool("allowhalfop");
-       ModeHandler* mh = ServerInstance->Modes->FindMode('h', MODETYPE_CHANNEL);
-       if (AllowHalfOp && !mh) {
-               ServerInstance->Logs->Log("CONFIG", DEFAULT, "Enabling halfop mode.");
-               mh = new ModeChannelHalfOp;
-               ServerInstance->Modes->AddMode(mh);
-       } else if (!AllowHalfOp && mh) {
-               ServerInstance->Logs->Log("CONFIG", DEFAULT, "Disabling halfop mode.");
-               ServerInstance->Modes->DelMode(mh);
-               delete mh;
-       }
-
        Module* whowas = ServerInstance->Modules->Find("cmd_whowas.so");
        if (whowas)
                WhowasRequest(NULL, whowas, WhowasRequest::WHOWAS_PRUNE).Send();
@@ -1143,6 +842,9 @@ void ServerConfig::ApplyModules(User* user)
                }
        }
 
+       if (ConfValue("options")->getBool("allowhalfop") && removed_modules.erase("m_halfop.so") == 0)
+               added_modules.push_back("m_halfop.so");
+
        for (std::set<std::string>::iterator removing = removed_modules.begin(); removing != removed_modules.end(); removing++)
        {
                // Don't remove cmd_*.so, just remove m_*.so
@@ -1194,10 +896,15 @@ bool ServerConfig::StartsWithWindowsDriveLetter(const std::string &path)
 
 ConfigTag* ServerConfig::ConfValue(const std::string &tag)
 {
-       ConfigDataHash::iterator iter = config_data.find(tag);
-       if (iter == config_data.end())
-               return NULL;
-       return iter->second;
+       ConfigTagList found = config_data.equal_range(tag);
+       if (found.first == found.second)
+               return EmptyTag;
+       ConfigTag* rv = found.first->second;
+       found.first++;
+       if (found.first != found.second)
+               ServerInstance->Logs->Log("CONFIG",DEFAULT, "Multiple <" + tag + "> tags found; only first will be used "
+                       "(first at " + rv->getTagLocation() + "; second at " + found.first->second->getTagLocation() + ")");
+       return rv;
 }
 
 ConfigTagList ServerConfig::ConfTags(const std::string& tag)
@@ -1205,121 +912,6 @@ ConfigTagList ServerConfig::ConfTags(const std::string& tag)
        return config_data.equal_range(tag);
 }
 
-bool ConfigTag::readString(const std::string& key, std::string& value, bool allow_lf)
-{
-       if (!this)
-               return false;
-       for(std::vector<KeyVal>::iterator j = items.begin(); j != items.end(); ++j)
-       {
-               if(j->first != key)
-                       continue;
-               value = j->second;
-               if (!allow_lf && (value.find('\n') != std::string::npos))
-               {
-                       ServerInstance->Logs->Log("CONFIG",DEFAULT, "Value of <" + tag + ":" + key + "> at " + getTagLocation() +
-                               " contains a linefeed, and linefeeds in this value are not permitted -- stripped to spaces.");
-                       for (std::string::iterator n = value.begin(); n != value.end(); n++)
-                               if (*n == '\n')
-                                       *n = ' ';
-               }
-               return true;
-       }
-       return false;
-}
-
-std::string ConfigTag::getString(const std::string& key, const std::string& def)
-{
-       std::string res = def;
-       readString(key, res);
-       return res;
-}
-
-long ConfigTag::getInt(const std::string &key, long def)
-{
-       std::string result;
-       if(!readString(key, result))
-               return def;
-
-       const char* res_cstr = result.c_str();
-       char* res_tail = NULL;
-       long res = strtol(res_cstr, &res_tail, 0);
-       if (res_tail == res_cstr)
-               return def;
-       switch (toupper(*res_tail))
-       {
-               case 'K':
-                       res= res* 1024;
-                       break;
-               case 'M':
-                       res= res* 1024 * 1024;
-                       break;
-               case 'G':
-                       res= res* 1024 * 1024 * 1024;
-                       break;
-       }
-       return res;
-}
-
-double ConfigTag::getFloat(const std::string &key, double def)
-{
-       std::string result;
-       if (!readString(key, result))
-               return def;
-       return strtod(result.c_str(), NULL);
-}
-
-bool ConfigTag::getBool(const std::string &key, bool def)
-{
-       std::string result;
-       if(!readString(key, result))
-               return def;
-
-       return (result == "yes" || result == "true" || result == "1" || result == "on");
-}
-
-std::string ConfigTag::getTagLocation()
-{
-       return src_name + ":" + ConvToStr(src_line);
-}
-
-std::string OperInfo::getConfig(const std::string& key)
-{
-       std::string rv;
-       if (type_block)
-               type_block->readString(key, rv);
-       if (oper_block)
-               oper_block->readString(key, rv);
-       return rv;
-}
-
-/** Read the contents of a file located by `fname' into a file_cache pointed at by `F'.
- */
-bool ServerConfig::ReadFile(file_cache &F, const std::string& fname)
-{
-       if (fname.empty())
-               return false;
-
-       char linebuf[MAXBUF];
-
-       F.clear();
-
-       FileWrapper file(fopen(fname.c_str(), "r"));
-
-       if (!file)
-               return false;
-       while (!feof(file))
-       {
-               if (fgets(linebuf, sizeof(linebuf), file))
-                       linebuf[strlen(linebuf)-1] = 0;
-               else
-                       *linebuf = 0;
-
-               F.push_back(*linebuf ? linebuf : " ");
-       }
-
-       return true;
-}
-
 bool ServerConfig::FileExists(const char* file)
 {
        struct stat sb;
@@ -1346,14 +938,13 @@ const char* ServerConfig::CleanFilename(const char* name)
        return (p != name ? ++p : p);
 }
 
-std::string ServerConfig::GetSID()
+const std::string& ServerConfig::GetSID()
 {
        return sid;
 }
 
 void ConfigReaderThread::Run()
 {
-       Config = new ServerConfig;
        Config->Read();
        done = true;
 }
@@ -1362,9 +953,7 @@ void ConfigReaderThread::Finish()
 {
        ServerConfig* old = ServerInstance->Config;
        ServerInstance->Logs->Log("CONFIG",DEBUG,"Switching to new configuration...");
-       ServerInstance->Logs->CloseLogs();
        ServerInstance->Config = this->Config;
-       ServerInstance->Logs->OpenFileLogs();
        Config->Apply(old, TheUserUID);
 
        if (Config->valid)
@@ -1375,24 +964,27 @@ void ConfigReaderThread::Finish()
                 * XXX: The order of these is IMPORTANT, do not reorder them without testing
                 * thoroughly!!!
                 */
-               ServerInstance->XLines->CheckELines();
+               ServerInstance->Users->RehashCloneCounts();
                ServerInstance->XLines->CheckELines();
                ServerInstance->XLines->ApplyLines();
                ServerInstance->Res->Rehash();
                ServerInstance->ResetMaxBans();
                Config->ApplyDisabledCommands(Config->DisabledCommands);
-               User* user = TheUserUID.empty() ? ServerInstance->FindNick(TheUserUID) : NULL;
+               User* user = ServerInstance->FindNick(TheUserUID);
                FOREACH_MOD(I_OnRehash, OnRehash(user));
                ServerInstance->BuildISupport();
 
-               delete old;
+               ServerInstance->Logs->CloseLogs();
+               ServerInstance->Logs->OpenFileLogs();
+
+               if (Config->RawLog && !old->RawLog)
+                       ServerInstance->Users->ServerNoticeAll("*** Raw I/O logging is enabled on this server. All messages, passwords, and commands are being recorded.");
+
+               Config = old;
        }
        else
        {
                // whoops, abort!
-               ServerInstance->Logs->CloseLogs();
                ServerInstance->Config = old;
-               ServerInstance->Logs->OpenFileLogs();
-               delete this->Config;
        }
 }