]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_spanningtree/treesocket1.cpp
Clean up duplicated link snomask messages on errors
[user/henk/code/inspircd.git] / src / modules / m_spanningtree / treesocket1.cpp
index 55fbc2c91e8e55cf925300ac4e743396cb944c05..fcd6871e1a06533b182221512070eb729ffdb7b6 100644 (file)
 #include "inspircd.h"
 #include "socket.h"
 #include "xline.h"
-#include "../transport.h"
 #include "../m_hash.h"
 #include "socketengine.h"
 
 #include "main.h"
+#include "../spanningtree.h"
 #include "utils.h"
 #include "treeserver.h"
 #include "link.h"
 #include "treesocket.h"
 #include "resolvers.h"
-#include "handshaketimer.h"
-
-/* $ModDep: m_spanningtree/resolvers.h m_spanningtree/main.h m_spanningtree/utils.h m_spanningtree/treeserver.h m_spanningtree/link.h m_spanningtree/treesocket.h m_hash.h m_spanningtree/handshaketimer.h */
-
 
 /** Because most of the I/O gubbins are encapsulated within
  * BufferedSocket, we just call the superclass constructor for
  * most of the action, and append a few of our own values
  * to it.
  */
-TreeSocket::TreeSocket(SpanningTreeUtilities* Util, std::string shost, int iport, unsigned long maxtime, const std::string &ServerName, const std::string &bindto, Autoconnect* myac, Module* HookMod)
+TreeSocket::TreeSocket(SpanningTreeUtilities* Util, const std::string& shost, int iport, unsigned long maxtime, const std::string &ServerName, const std::string &bindto, Autoconnect* myac, const std::string& hook)
        : Utils(Util), IP(shost), myautoconnect(myac)
 {
        age = ServerInstance->Time();
@@ -42,35 +38,53 @@ TreeSocket::TreeSocket(SpanningTreeUtilities* Util, std::string shost, int iport
        capab_phase = 0;
        proto_version = 0;
        LinkState = CONNECTING;
+       if (!hook.empty())
+       {
+               modulelist* ml = ServerInstance->Modules->FindInterface("BufferedSocketHook");
+               if (ml)
+               {
+                       for(modulelist::iterator i = ml->begin(); i != ml->end(); ++i)
+                       {
+                               std::string name = (**i).ModuleSourceFile;
+                               int a = name.rfind('_');
+                               int b = name.rfind('.');
+                               name = name.substr(a+1, b-a-1);
+                               if (name == hook)
+                               {
+                                       AddIOHook(*i);
+                                       goto found;
+                               }
+                       }
+               }
+               SetError("Could not find hook '" + hook + "' for connection to " + ServerName);
+               return;
+       }
+found:
        DoConnect(shost, iport, maxtime, bindto);
        Utils->timeoutlist[this] = std::pair<std::string, int>(ServerName, maxtime);
-       if (HookMod)
-               BufferedSocketHookRequest(this, Utils->Creator, HookMod).Send();
-       hstimer = NULL;
+       SendCapabilities(1);
 }
 
 /** When a listening socket gives us a new file descriptor,
  * we must associate it with a socket without creating a new
  * connection. This constructor is used for this purpose.
  */
-TreeSocket::TreeSocket(SpanningTreeUtilities* Util, int newfd, char* ip, Autoconnect* myac, Module* HookMod)
-       : BufferedSocket(newfd), Utils(Util), IP(ip), myautoconnect(myac)
+TreeSocket::TreeSocket(SpanningTreeUtilities* Util, int newfd, ListenSocket* via, irc::sockets::sockaddrs* client, irc::sockets::sockaddrs* server)
+       : BufferedSocket(newfd), Utils(Util)
 {
+       IP = client->addr();
        age = ServerInstance->Time();
        LinkState = WAIT_AUTH_1;
        capab_phase = 0;
        proto_version = 0;
-       /* If we have a transport module hooked to the parent, hook the same module to this
-        * socket, and set a timer waiting for handshake before we send CAPAB etc.
-        */
-       if (HookMod)
-               BufferedSocketHookRequest(this, Utils->Creator, HookMod).Send();
+       myhost = "inbound from " + IP;
 
-       hstimer = new HandshakeTimer(ServerInstance, this, &(Utils->LinkBlocks[0]), this->Utils, 1);
-       ServerInstance->Timers->AddTimer(hstimer);
+       FOREACH_MOD(I_OnHookIO, OnHookIO(this, via));
+       if (GetIOHook())
+               GetIOHook()->OnStreamSocketAccept(this, client, server);
+       SendCapabilities(1);
 
-       /* Fix by Brain - inbound sockets need a timeout, too. 30 secs should be pleanty */
-       Utils->timeoutlist[this] = std::pair<std::string, int>("<unknown>", 30);
+       Utils->timeoutlist[this] = std::pair<std::string, int>(myhost, 30);
 }
 
 ServerState TreeSocket::GetLinkState()
@@ -88,13 +102,16 @@ void TreeSocket::CleanNegotiationInfo()
        OutboundPass.clear();
 }
 
-TreeSocket::~TreeSocket()
+CullResult TreeSocket::cull()
 {
-       if (GetIOHook())
-               BufferedSocketUnhookRequest(this, Utils->Creator, GetIOHook()).Send();
-       if (hstimer)
-               ServerInstance->Timers->DelTimer(hstimer);
        Utils->timeoutlist.erase(this);
+       if (myautoconnect)
+               Utils->Creator->ConnectServer(myautoconnect, false);
+       return this->BufferedSocket::cull();
+}
+
+TreeSocket::~TreeSocket()
+{
 }
 
 /** When an outbound connection finishes connecting, we receive
@@ -108,20 +125,14 @@ void TreeSocket::OnConnected()
        if (this->LinkState == CONNECTING)
        {
                /* we do not need to change state here. */
-               for (std::vector<Link>::iterator x = Utils->LinkBlocks.begin(); x < Utils->LinkBlocks.end(); x++)
+               for (std::vector<reference<Link> >::iterator i = Utils->LinkBlocks.begin(); i < Utils->LinkBlocks.end(); ++i)
                {
+                       Link* x = *i;
                        if (x->Name == this->myhost)
                        {
-                               ServerInstance->SNO->WriteToSnoMask('l', "Connection to \2%s\2[%s] started.", myhost.c_str(), (x->HiddenFromStats ? "<hidden>" : this->IP.c_str()));
+                               ServerInstance->SNO->WriteGlobalSno('l', "Connection to \2%s\2[%s] started.", myhost.c_str(), (x->HiddenFromStats ? "<hidden>" : this->IP.c_str()));
                                this->OutboundPass = x->SendPass;
-                               if (GetIOHook())
-                               {
-                                       ServerInstance->SNO->WriteToSnoMask('l', "Connection to \2%s\2[%s] using transport \2%s\2", myhost.c_str(), (x->HiddenFromStats ? "<hidden>" : this->IP.c_str()), x->Hook.c_str());
-                                       hstimer = new HandshakeTimer(ServerInstance, this, &(*x), this->Utils, 1);
-                                       ServerInstance->Timers->AddTimer(hstimer);
-                               }
-                               else
-                                       this->SendCapabilities(1);
+                               this->SendCapabilities(1);
                                return;
                        }
                }
@@ -131,41 +142,19 @@ void TreeSocket::OnConnected()
         * If that happens the connection hangs here until it's closed. Unlikely
         * and rather harmless.
         */
-       ServerInstance->SNO->WriteToSnoMask('l', "Connection to \2%s\2 lost link tag(!)", myhost.c_str());
+       ServerInstance->SNO->WriteGlobalSno('l', "Connection to \2%s\2 lost link tag(!)", myhost.c_str());
 }
 
 void TreeSocket::OnError(BufferedSocketError e)
 {
-       switch (e)
-       {
-               case I_ERR_CONNECT:
-                       ServerInstance->SNO->WriteToSnoMask('l', "Connection failed: Connection to \002%s\002 refused", myhost.c_str());
-                       Utils->DoFailOver(myautoconnect);
-               break;
-               case I_ERR_SOCKET:
-                       ServerInstance->SNO->WriteToSnoMask('l', "Connection failed: Could not create socket (%s)", strerror(errno));
-               break;
-               case I_ERR_BIND:
-                       ServerInstance->SNO->WriteToSnoMask('l', "Connection failed: Error binding socket to address or port (%s)", strerror(errno));
-               break;
-               case I_ERR_WRITE:
-                       ServerInstance->SNO->WriteToSnoMask('l', "Connection failed: I/O error on connection (%s)", errno ? strerror(errno) : "Connection closed unexpectedly");
-               break;
-               case I_ERR_NOMOREFDS:
-                       ServerInstance->SNO->WriteToSnoMask('l', "Connection failed: Operating system is out of file descriptors!");
-               break;
-               default:
-                       if ((errno) && (errno != EINPROGRESS) && (errno != EAGAIN))
-                               ServerInstance->SNO->WriteToSnoMask('l', "Connection to \002%s\002 failed with OS error: %s", myhost.c_str(), strerror(errno));
-               break;
-       }
+       ServerInstance->SNO->WriteGlobalSno('l', "Connection to \002%s\002 failed with error: %s",
+               myhost.c_str(), getError().c_str());
 }
 
 void TreeSocket::SendError(const std::string &errormessage)
 {
-       /* Display the error locally as well as sending it remotely */
-       ServerInstance->SNO->WriteToSnoMask('l', "Sent \2ERROR\2 to %s: %s", (this->InboundServerName.empty() ? this->IP.c_str() : this->InboundServerName.c_str()), errormessage.c_str());
        WriteLine("ERROR :"+errormessage);
+       SetError(errormessage);
 }
 
 /** This function forces this server to quit, removing this server
@@ -176,6 +165,8 @@ void TreeSocket::SendError(const std::string &errormessage)
  */
 void TreeSocket::SquitServer(std::string &from, TreeServer* Current)
 {
+       ServerInstance->Logs->Log("m_spanningtree",DEBUG,"SquitServer for %s from %s",
+               Current->GetName().c_str(), from.c_str());
        /* recursively squit the servers attached to 'Current'.
         * We're going backwards so we don't remove users
         * while we still need them ;)
@@ -200,8 +191,7 @@ void TreeSocket::Squit(TreeServer* Current, const std::string &reason)
 
        if ((Current) && (Current != Utils->TreeRoot))
        {
-               Event rmode((char*)Current->GetName().c_str(), (Module*)Utils->Creator, "lost_server");
-               rmode.Send(ServerInstance);
+               DelServerEvent(Utils->Creator, Current->GetName());
 
                parameterlist params;
                params.push_back(Current->GetName());
@@ -209,12 +199,12 @@ void TreeSocket::Squit(TreeServer* Current, const std::string &reason)
                Utils->DoOneToAllButSender(Current->GetParent()->GetName(),"SQUIT",params,Current->GetName());
                if (Current->GetParent() == Utils->TreeRoot)
                {
-                       ServerInstance->SNO->WriteToSnoMask('l', "Server \002"+Current->GetName()+"\002 split: "+reason);
+                       ServerInstance->SNO->WriteGlobalSno('l', "Server \002"+Current->GetName()+"\002 split: "+reason);
                        LocalSquit = true;
                }
                else
                {
-                       ServerInstance->SNO->WriteToSnoMask('L', "Server \002"+Current->GetName()+"\002 split from server \002"+Current->GetParent()->GetName()+"\002 with reason: "+reason);
+                       ServerInstance->SNO->WriteGlobalSno('L', "Server \002"+Current->GetName()+"\002 split from server \002"+Current->GetParent()->GetName()+"\002 with reason: "+reason);
                }
                num_lost_servers = 0;
                num_lost_users = 0;
@@ -222,11 +212,12 @@ void TreeSocket::Squit(TreeServer* Current, const std::string &reason)
                SquitServer(from, Current);
                Current->Tidy();
                Current->GetParent()->DelChild(Current);
+               Current->cull();
                delete Current;
                if (LocalSquit)
-                       ServerInstance->SNO->WriteToSnoMask('l', "Netsplit complete, lost \002%d\002 users on \002%d\002 servers.", num_lost_users, num_lost_servers);
+                       ServerInstance->SNO->WriteToSnoMask('l', "Netsplit complete, lost \002%d\002 user%s on \002%d\002 server%s.", num_lost_users, num_lost_users != 1 ? "s" : "", num_lost_servers, num_lost_servers != 1 ? "s" : "");
                else
-                       ServerInstance->SNO->WriteToSnoMask('L', "Netsplit complete, lost \002%d\002 users on \002%d\002 servers.", num_lost_users, num_lost_servers);
+                       ServerInstance->SNO->WriteToSnoMask('L', "Netsplit complete, lost \002%d\002 user%s on \002%d\002 server%s.", num_lost_users, num_lost_users != 1 ? "s" : "", num_lost_servers, num_lost_servers != 1 ? "s" : "");
        }
        else
                ServerInstance->Logs->Log("m_spanningtree",DEFAULT,"Squit from unknown server");
@@ -251,14 +242,7 @@ void TreeSocket::OnDataReady()
                 */
                if (ret.find("\r") != std::string::npos)
                        ret = recvq.substr(0,recvq.find("\r")-1);
-               /* Process this one, abort if it
-                * didnt return true.
-                */
-               if (!this->ProcessLine(ret))
-               {
-                       SetError("ProcessLine returned false");
-                       break;
-               }
+               ProcessLine(ret);
        }
        Utils->Creator->loopCall = false;
 }