m_spanningtree Ignore incoming BURST command in CONNECTED state

[user/henk/code/inspircd.git] / src / modules / m_spanningtree / treeserver.h

diff --git a/src/modules/m_spanningtree/treeserver.h b/src/modules/m_spanningtree/treeserver.h
index 4465de15e4ea439563a86e0f4b3eae6eed2b4aab..b7e9ee9d9bdd865d4de2a09ec54346acbf84dea9 100644 (file)
--- a/src/modules/m_spanningtree/treeserver.h
+++ b/src/modules/m_spanningtree/treeserver.h
@@ -22,6 +22,7 @@
 #pragma once
 
 #include "treesocket.h"
+#include "pingtimer.h"
 
 /** Each server in the tree is represented by one class of
  * type TreeServer. A locally connected TreeServer can
@@ -49,8 +50,6 @@ class TreeServer : public Server
        std::string fullversion;
 
        TreeSocket* Socket;                     /* Socket used to communicate with this server */
-       time_t NextPing;                        /* After this time, the server should be PINGed*/
-       bool LastPingWasGood;                   /* True if the server responded to the last PING with a PONG */
        std::string sid;                        /* Server ID */
 
        /** Counter counting how many servers are bursting in front of this server, including
@@ -64,6 +63,10 @@ class TreeServer : public Server
         */
        bool isdead;
 
+       /** Timer handling PINGing the server and killing it on timeout
+        */
+       PingTimer pingtimer;
+
        /** This method is used to add this TreeServer to the
         * hash maps. It is only called by the constructors.
         */
@@ -82,8 +85,6 @@ class TreeServer : public Server
        FakeUser* const ServerUser;             /* User representing this server */
        const time_t age;
 
-       bool Warned;                            /* True if we've warned opers about high latency on this server */
-
        unsigned int UserCount;                 /* How many users are on this server? [note: doesn't care about +i] */
        unsigned int OperCount;                 /* How many opers are on this server? */
 
@@ -143,38 +144,18 @@ class TreeServer : public Server
         */
        const std::string& GetFullVersion() const { return fullversion; }
 
-       /** Set time we are next due to ping this server
-        */
-       void SetNextPingTime(time_t t);
-
-       /** Get the time we are next due to ping this server
-        */
-       time_t NextPingTime();
-
-       /** Last ping time in milliseconds, used to calculate round trip time
-        */
-       unsigned long LastPingMsec;
-
        /** Round trip time of last ping
         */
        unsigned long rtt;
 
        /** When we recieved BURST from this server, used to calculate total burst time at ENDBURST.
         */
-       unsigned long StartBurst;
+       uint64_t StartBurst;
 
        /** True if this server is hidden
         */
        bool Hidden;
 
-       /** True if the server answered their last ping
-        */
-       bool AnsweredLastPing();
-
-       /** Set the server as responding to its last ping
-        */
-       void SetPingFlag();
-
        /** Get the TreeSocket pointer for local servers.
         * For remote servers, this returns NULL.
         */
@@ -232,7 +213,11 @@ class TreeServer : public Server
        /** Set the bursting state of the server
         * @param startms Time the server started bursting, if 0 or omitted, use current time
         */
-       void BeginBurst(unsigned long startms = 0);
+       void BeginBurst(uint64_t startms = 0);
+
+       /** Register a PONG from the server
+        */
+       void OnPong() { pingtimer.OnPong(); }
 
        CullResult cull();