]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_cgiirc.cpp
Rename the cmd variable to cmdwebirc in the cgiirc module.
[user/henk/code/inspircd.git] / src / modules / m_cgiirc.cpp
index df8201bde39b4360281ead891722234ecc24bb6c..107824985f913b01a5cb02eeeb931c893be0b20f 100644 (file)
@@ -1,13 +1,18 @@
 /*
  * InspIRCd -- Internet Relay Chat Daemon
  *
+ *   Copyright (C) 2019 linuxdaemon <linuxdaemon.irc@gmail.com>
+ *   Copyright (C) 2014 md_5 <git@md-5.net>
+ *   Copyright (C) 2014 Googolplexed <googol@googolplexed.net>
+ *   Copyright (C) 2013, 2017-2018, 2020 Sadie Powell <sadie@witchery.services>
+ *   Copyright (C) 2013 Adam <Adam@anope.org>
+ *   Copyright (C) 2012-2013, 2015 Attila Molnar <attilamolnar@hush.com>
+ *   Copyright (C) 2012, 2019 Robby <robby@chatbelgie.be>
  *   Copyright (C) 2009-2010 Daniel De Graaf <danieldg@inspircd.org>
- *   Copyright (C) 2007-2008 John Brooks <john.brooks@dereferenced.net>
- *   Copyright (C) 2008 Pippijn van Steenhoven <pip88nl@gmail.com>
- *   Copyright (C) 2006-2008 Craig Edwards <craigedwards@brainbox.cc>
- *   Copyright (C) 2007 Robin Burchell <robin+git@viroteck.net>
+ *   Copyright (C) 2009 Uli Schlachter <psychon@inspircd.org>
+ *   Copyright (C) 2007-2009 Robin Burchell <robin+git@viroteck.net>
  *   Copyright (C) 2007 Dennis Friis <peavey@inspircd.org>
- *   Copyright (C) 2006 Oliver Lupton <oliverlupton@gmail.com>
+ *   Copyright (C) 2006-2007, 2010 Craig Edwards <brain@inspircd.org>
  *
  * 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
@@ -34,16 +39,19 @@ enum
        RPL_WHOISGATEWAY = 350
 };
 
+// One or more hostmask globs or CIDR ranges.
+typedef std::vector<std::string> MaskList;
+
 // Encapsulates information about an ident host.
 class IdentHost
 {
  private:
-       std::string hostmask;
+       MaskList hostmasks;
        std::string newident;
 
  public:
-       IdentHost(const std::string& mask, const std::string& ident)
-               : hostmask(mask)
+       IdentHost(const MaskList& masks, const std::string& ident)
+               : hostmasks(masks)
                , newident(ident)
        {
        }
@@ -55,10 +63,19 @@ class IdentHost
 
        bool Matches(LocalUser* user) const
        {
-               if (!InspIRCd::Match(user->GetRealHost(), hostmask, ascii_case_insensitive_map))
-                       return false;
+               for (MaskList::const_iterator iter = hostmasks.begin(); iter != hostmasks.end(); ++iter)
+               {
+                       // Does the user's hostname match this hostmask?
+                       if (InspIRCd::Match(user->GetRealHost(), *iter, ascii_case_insensitive_map))
+                               return true;
 
-               return InspIRCd::MatchCIDR(user->GetIPString(), hostmask, ascii_case_insensitive_map);
+                       // Does the user's IP address match this hostmask?
+                       if (InspIRCd::MatchCIDR(user->GetIPString(), *iter, ascii_case_insensitive_map))
+                               return true;
+               }
+
+               // The user didn't match any hostmasks.
+               return false;
        }
 };
 
@@ -66,14 +83,14 @@ class IdentHost
 class WebIRCHost
 {
  private:
-       std::string hostmask;
+       MaskList hostmasks;
        std::string fingerprint;
        std::string password;
        std::string passhash;
 
  public:
-       WebIRCHost(const std::string& mask, const std::string& fp, const std::string& pass, const std::string& hash)
-               : hostmask(mask)
+       WebIRCHost(const MaskList& masks, const std::string& fp, const std::string& pass, const std::string& hash)
+               : hostmasks(masks)
                , fingerprint(fp)
                , password(pass)
                , passhash(hash)
@@ -91,26 +108,22 @@ class WebIRCHost
                if (!fingerprint.empty() && !InspIRCd::TimingSafeCompare(fp, fingerprint))
                        return false;
 
-               // Does the user's hostname match our hostmask?
-               if (InspIRCd::Match(user->GetRealHost(), hostmask, ascii_case_insensitive_map))
-                       return true;
+               for (MaskList::const_iterator iter = hostmasks.begin(); iter != hostmasks.end(); ++iter)
+               {
+                       // Does the user's hostname match this hostmask?
+                       if (InspIRCd::Match(user->GetRealHost(), *iter, ascii_case_insensitive_map))
+                               return true;
 
-               // Does the user's IP address match our hostmask?
-               return InspIRCd::MatchCIDR(user->GetIPString(), hostmask, ascii_case_insensitive_map);
+                       // Does the user's IP address match this hostmask?
+                       if (InspIRCd::MatchCIDR(user->GetIPString(), *iter, ascii_case_insensitive_map))
+                               return true;
+               }
+
+               // The user didn't match any hostmasks.
+               return false;
        }
 };
 
-/*
- * WEBIRC
- *  This is used for the webirc method of CGIIRC auth, and is (really) the best way to do these things.
- *  Syntax: WEBIRC password gateway hostname ip
- *  Where password is a shared key, gateway is the name of the WebIRC gateway and version (e.g. cgiirc), hostname
- *  is the resolved host of the client issuing the command and IP is the real IP of the client.
- *
- * How it works:
- *  To tie in with the rest of cgiirc module, and to avoid race conditions, /webirc is only processed locally
- *  and simply sets metadata on the user, which is later decoded on full connect to give something meaningful.
- */
 class CommandWebIRC : public SplitCommand
 {
  public:
@@ -132,7 +145,7 @@ class CommandWebIRC : public SplitCommand
        {
                allow_empty_last_param = false;
                works_before_reg = true;
-               this->syntax = "<password> <gateway> <hostname> <ip> [flags]";
+               this->syntax = "<password> <gateway> <hostname> <ip> [<flags>]";
        }
 
        CmdResult HandleLocal(LocalUser* user, const Params& parameters) CXX11_OVERRIDE
@@ -160,8 +173,9 @@ class CommandWebIRC : public SplitCommand
                        realhost.set(user, user->GetRealHost());
                        realip.set(user, user->GetIPString());
 
-                       WriteLog("Connecting user %s is using a WebIRC gateway; changing their IP from %s to %s.",
-                               user->uuid.c_str(), user->GetIPString().c_str(), parameters[3].c_str());
+                       WriteLog("Connecting user %s is using the %s WebIRC gateway; changing their IP from %s to %s.",
+                               user->uuid.c_str(), parameters[1].c_str(),
+                               user->GetIPString().c_str(), parameters[3].c_str());
 
                        // If we have custom flags then deal with them.
                        WebIRC::FlagMap flags;
@@ -222,7 +236,7 @@ class ModuleCgiIRC
        , public Whois::EventListener
 {
  private:
-       CommandWebIRC cmd;
+       CommandWebIRC cmdwebirc;
        std::vector<IdentHost> hosts;
 
        static bool ParseIdent(LocalUser* user, irc::sockets::sockaddrs& out)
@@ -265,7 +279,7 @@ class ModuleCgiIRC
        ModuleCgiIRC()
                : WebIRC::EventListener(this)
                , Whois::EventListener(this)
-               , cmd(this)
+               , cmdwebirc(this)
        {
        }
 
@@ -284,9 +298,13 @@ class ModuleCgiIRC
                {
                        ConfigTag* tag = i->second;
 
+                       MaskList masks;
+                       irc::spacesepstream maskstream(tag->getString("mask"));
+                       for (std::string mask; maskstream.GetToken(mask); )
+                               masks.push_back(mask);
+
                        // Ensure that we have the <cgihost:mask> parameter.
-                       const std::string mask = tag->getString("mask");
-                       if (mask.empty())
+                       if (masks.empty())
                                throw ModuleException("<cgihost:mask> is a mandatory field, at " + tag->getTagLocation());
 
                        // Determine what lookup type this host uses.
@@ -295,32 +313,39 @@ class ModuleCgiIRC
                        {
                                // The IP address should be looked up from the hex IP address.
                                const std::string newident = tag->getString("newident", "gateway", ServerInstance->IsIdent);
-                               identhosts.push_back(IdentHost(mask, newident));
+                               identhosts.push_back(IdentHost(masks, newident));
                        }
                        else if (stdalgo::string::equalsci(type, "webirc"))
                        {
                                // The IP address will be received via the WEBIRC command.
                                const std::string fingerprint = tag->getString("fingerprint");
                                const std::string password = tag->getString("password");
+                               const std::string passwordhash = tag->getString("hash", "plaintext", 1);
 
                                // WebIRC blocks require a password.
                                if (fingerprint.empty() && password.empty())
                                        throw ModuleException("When using <cgihost type=\"webirc\"> either the fingerprint or password field is required, at " + tag->getTagLocation());
 
-                               webirchosts.push_back(WebIRCHost(mask, fingerprint, password, tag->getString("hash")));
+                               if (!password.empty() && stdalgo::string::equalsci(passwordhash, "plaintext"))
+                               {
+                                       ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "<cgihost> tag at %s contains an plain text password, this is insecure!",
+                                               tag->getTagLocation().c_str());
+                               }
+
+                               webirchosts.push_back(WebIRCHost(masks, fingerprint, password, passwordhash));
                        }
                        else
                        {
-                               throw ModuleException(type + " is an invalid <cgihost:mask> type, at " + tag->getTagLocation()); 
+                               throw ModuleException(type + " is an invalid <cgihost:mask> type, at " + tag->getTagLocation());
                        }
                }
 
                // The host configuration was valid so we can apply it.
                hosts.swap(identhosts);
-               cmd.hosts.swap(webirchosts);
+               cmdwebirc.hosts.swap(webirchosts);
 
                // Do we send an oper notice when a m_cgiirc client has their IP changed?
-               cmd.notify = ServerInstance->Config->ConfValue("cgiirc")->getBool("opernotice", true);
+               cmdwebirc.notify = ServerInstance->Config->ConfValue("cgiirc")->getBool("opernotice", true);
        }
 
        ModResult OnSetConnectClass(LocalUser* user, ConnectClass* myclass) CXX11_OVERRIDE
@@ -332,19 +357,30 @@ class ModuleCgiIRC
 
                // If the user is not connecting via a WebIRC gateway then they
                // cannot match this connect class.
-               const std::string* gateway = cmd.gateway.get(user);
+               const std::string* gateway = cmdwebirc.gateway.get(user);
                if (!gateway)
+               {
+                       ServerInstance->Logs->Log("CONNECTCLASS", LOG_DEBUG, "The %s connect class is not suitable as it requires a connection via a WebIRC gateway",
+                                       myclass->GetName().c_str());
                        return MOD_RES_DENY;
+               }
 
                // If the gateway matches the <connect:webirc> constraint then
                // allow the check to continue. Otherwise, reject it.
-               return InspIRCd::Match(*gateway, webirc) ? MOD_RES_PASSTHRU : MOD_RES_DENY;
+               if (!InspIRCd::Match(*gateway, webirc))
+               {
+                       ServerInstance->Logs->Log("CONNECTCLASS", LOG_DEBUG, "The %s connect class is not suitable as the WebIRC gateway name (%s) does not match %s",
+                                       myclass->GetName().c_str(), gateway->c_str(), webirc.c_str());
+                       return MOD_RES_DENY;
+               }
+
+               return MOD_RES_PASSTHRU;
        }
 
        ModResult OnUserRegister(LocalUser* user) CXX11_OVERRIDE
        {
                // There is no need to check for gateways if one is already being used.
-               if (cmd.realhost.get(user))
+               if (cmdwebirc.realhost.get(user))
                        return MOD_RES_PASSTHRU;
 
                for (std::vector<IdentHost>::const_iterator iter = hosts.begin(); iter != hosts.end(); ++iter)
@@ -360,16 +396,16 @@ class ModuleCgiIRC
                                return MOD_RES_PASSTHRU;
 
                        // Store the hostname and IP of the gateway for later use.
-                       cmd.realhost.set(user, user->GetRealHost());
-                       cmd.realip.set(user, user->GetIPString());
+                       cmdwebirc.realhost.set(user, user->GetRealHost());
+                       cmdwebirc.realip.set(user, user->GetIPString());
 
                        const std::string& newident = iter->GetIdent();
-                       cmd.WriteLog("Connecting user %s is using an ident gateway; changing their IP from %s to %s and their ident from %s to %s.",
+                       cmdwebirc.WriteLog("Connecting user %s is using an ident gateway; changing their IP from %s to %s and their ident from %s to %s.",
                                user->uuid.c_str(), user->GetIPString().c_str(), address.addr().c_str(), user->ident.c_str(), newident.c_str());
 
                        user->ChangeIdent(newident);
                        user->SetClientIP(address);
-                       break; 
+                       break;
                }
                return MOD_RES_PASSTHRU;
        }
@@ -440,12 +476,12 @@ class ModuleCgiIRC
                        return;
 
                // If these fields are not set then the client is not using a gateway.
-               const std::string* realhost = cmd.realhost.get(whois.GetTarget());
-               const std::string* realip = cmd.realip.get(whois.GetTarget());
+               const std::string* realhost = cmdwebirc.realhost.get(whois.GetTarget());
+               const std::string* realip = cmdwebirc.realip.get(whois.GetTarget());
                if (!realhost || !realip)
                        return;
 
-               const std::string* gateway = cmd.gateway.get(whois.GetTarget());
+               const std::string* gateway = cmdwebirc.gateway.get(whois.GetTarget());
                if (gateway)
                        whois.SendLine(RPL_WHOISGATEWAY, *realhost, *realip, "is connected via the " + *gateway + " WebIRC gateway");
                else
@@ -454,7 +490,7 @@ class ModuleCgiIRC
 
        Version GetVersion() CXX11_OVERRIDE
        {
-               return Version("Enables forwarding the real IP address of a user from a gateway to the IRC server", VF_VENDOR);
+               return Version("Adds the ability for IRC gateways to forward the real IP address of users connecting through them.", VF_VENDOR);
        }
 };