]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_hostchange.cpp
Merge insp20
[user/henk/code/inspircd.git] / src / modules / m_hostchange.cpp
index 2635015c7c242619fa4328ded7fc750a68fa1aa8..6d5896ef51bd73ada8bedd4d875224030cc6a613 100644 (file)
 
 #include "inspircd.h"
 
-/* $ModDesc: Provides masking of user hostnames in a different way to m_cloaking */
-
 /** Holds information on a host set by m_hostchange
  */
 class Host
 {
  public:
-       std::string action;
+       enum HostChangeAction
+       {
+               HCA_SET,
+               HCA_SUFFIX,
+               HCA_ADDNICK
+       };
+
+       HostChangeAction action;
        std::string newhost;
        std::string ports;
+
+       Host(HostChangeAction Action, const std::string& Newhost, const std::string& Ports) :
+               action(Action), newhost(Newhost), ports(Ports) {}
 };
 
-typedef std::map<std::string,Host*> hostchanges_t;
+typedef std::vector<std::pair<std::string, Host> > hostchanges_t;
 
 class ModuleHostChange : public Module
 {
- private:
        hostchanges_t hostchanges;
        std::string MySuffix;
        std::string MyPrefix;
        std::string MySeparator;
 
  public:
-       ModuleHostChange()
-                       {
-               OnRehash(NULL);
-               Implementation eventlist[] = { I_OnRehash, I_OnUserConnect };
-               ServerInstance->Modules->Attach(eventlist, this, 2);
-       }
-
-       virtual ~ModuleHostChange()
+       void ReadConfig(ConfigStatus& status) CXX11_OVERRIDE
        {
-               for (hostchanges_t::iterator i = hostchanges.begin(); i != hostchanges.end(); i++)
-               {
-                       delete i->second;
-               }
+               ConfigTag* host = ServerInstance->Config->ConfValue("host");
+               MySuffix = host->getString("suffix");
+               MyPrefix = host->getString("prefix");
+               MySeparator = host->getString("separator", ".");
                hostchanges.clear();
-       }
 
-       void Prioritize()
-       {
-               Module* cloak = ServerInstance->Modules->Find("m_cloaking.so");
-               ServerInstance->Modules->SetPriority(this, I_OnUserConnect, PRIORITY_AFTER, &cloak);
-       }
-
-
-       virtual void OnRehash(User* user)
-       {
-               ConfigReader Conf;
-               MySuffix = Conf.ReadValue("host","suffix",0);
-               MyPrefix = Conf.ReadValue("host","prefix","",0);
-               MySeparator = Conf.ReadValue("host","separator",".",0);
-               for (hostchanges_t::iterator i = hostchanges.begin(); i != hostchanges.end(); i++)
+               std::set<std::string> dupecheck;
+               ConfigTagList tags = ServerInstance->Config->ConfTags("hostchange");
+               for (ConfigIter i = tags.first; i != tags.second; ++i)
                {
-                       delete i->second;
-               }
-               hostchanges.clear();
-               for (int index = 0; index < Conf.Enumerate("hostchange"); index++)
-               {
-                       std::string mask = Conf.ReadValue("hostchange", "mask", index);
-                       std::string ports = Conf.ReadValue("hostchange", "ports", index);
-                       std::string action = Conf.ReadValue("hostchange", "action", index);
-                       std::string newhost = Conf.ReadValue("hostchange", "value", index);
-                       Host* x = new Host;
-                       x->action = action;
-                       x->ports = ports;
-                       x->newhost = newhost;
-                       hostchanges[mask] = x;
+                       ConfigTag* tag = i->second;
+                       std::string mask = tag->getString("mask");
+                       if (!dupecheck.insert(mask).second)
+                               throw ModuleException("Duplicate hostchange entry: " + mask);
+
+                       Host::HostChangeAction act;
+                       std::string newhost;
+                       std::string action = tag->getString("action");
+                       if (!strcasecmp(action.c_str(), "set"))
+                       {
+                               act = Host::HCA_SET;
+                               newhost = tag->getString("value");
+                       }
+                       else if (!strcasecmp(action.c_str(), "suffix"))
+                               act = Host::HCA_SUFFIX;
+                       else if (!strcasecmp(action.c_str(), "addnick"))
+                               act = Host::HCA_ADDNICK;
+                       else
+                               throw ModuleException("Invalid hostchange action: " + action);
+
+                       hostchanges.push_back(std::make_pair(mask, Host(act, newhost, tag->getString("ports"))));
                }
        }
 
-       virtual Version GetVersion()
+       Version GetVersion() CXX11_OVERRIDE
        {
                // returns the version number of the module to be
                // listed in /MODULES
                return Version("Provides masking of user hostnames in a different way to m_cloaking", VF_VENDOR);
        }
 
-       virtual void OnUserConnect(LocalUser* user)
+       void OnUserConnect(LocalUser* user) CXX11_OVERRIDE
        {
                for (hostchanges_t::iterator i = hostchanges.begin(); i != hostchanges.end(); i++)
                {
                        if (((InspIRCd::MatchCIDR(user->MakeHost(), i->first)) || (InspIRCd::MatchCIDR(user->MakeHostIP(), i->first))))
                        {
-                               Host* h = i->second;
+                               const Host& h = i->second;
 
-                               if (!h->ports.empty())
+                               if (!h.ports.empty())
                                {
-                                       irc::portparser portrange(h->ports, false);
+                                       irc::portparser portrange(h.ports, false);
                                        long portno = -1;
                                        bool foundany = false;
 
@@ -123,27 +118,26 @@ class ModuleHostChange : public Module
 
                                // host of new user matches a hostchange tag's mask
                                std::string newhost;
-                               if (h->action == "set")
+                               if (h.action == Host::HCA_SET)
                                {
-                                       newhost = h->newhost;
+                                       newhost = h.newhost;
                                }
-                               else if (h->action == "suffix")
+                               else if (h.action == Host::HCA_SUFFIX)
                                {
                                        newhost = MySuffix;
                                }
-                               else if (h->action == "addnick")
+                               else if (h.action == Host::HCA_ADDNICK)
                                {
                                        // first take their nick and strip out non-dns, leaving just [A-Z0-9\-]
                                        std::string complete;
-                                       std::string old = user->nick;
-                                       for (unsigned int j = 0; j < old.length(); j++)
+                                       for (std::string::const_iterator j = user->nick.begin(); j != user->nick.end(); ++j)
                                        {
-                                               if  (((old[j] >= 'A') && (old[j] <= 'Z')) ||
-                                                   ((old[j] >= 'a') && (old[j] <= 'z')) ||
-                                                   ((old[j] >= '0') && (old[j] <= '9')) ||
-                                                   (old[j] == '-'))
+                                               if  (((*j >= 'A') && (*j <= 'Z')) ||
+                                                   ((*j >= 'a') && (*j <= 'z')) ||
+                                                   ((*j >= '0') && (*j <= '9')) ||
+                                                   (*j == '-'))
                                                {
-                                                       complete = complete + old[j];
+                                                       complete = complete + *j;
                                                }
                                        }
                                        if (complete.empty())
@@ -156,9 +150,9 @@ class ModuleHostChange : public Module
                                }
                                if (!newhost.empty())
                                {
-                                       user->WriteServ("NOTICE "+std::string(user->nick)+" :Setting your virtual host: " + newhost);
-                                       if (!user->ChangeDisplayedHost(newhost.c_str()))
-                                               user->WriteServ("NOTICE "+std::string(user->nick)+" :Could not set your virtual host: " + newhost);
+                                       user->WriteNotice("Setting your virtual host: " + newhost);
+                                       if (!user->ChangeDisplayedHost(newhost))
+                                               user->WriteNotice("Could not set your virtual host: " + newhost);
                                        return;
                                }
                        }