Refactor node to allow full_node to include sessions.

Author: evoskuil

include/bitcoin/node/define.hpp

@@ -120,10 +120,9 @@ using type_id = network::messages::peer::inventory_item::type_id;
 // settings       : define
 // configuration  : define settings
 // parser         : define configuration
-// /chasers       : define configuration  [forward: full_node]
+// /chasers       : define configuration     [forward: full_node]
 // full_node      : define /chasers
-// session        : define full_node
-// /protocols     : define session
-
-// Session is only included by full_node.cpp (avoids cycle).
-// /sessions      : define full_node /protocols
+// attach         : define session           [forward: full_node]
+// session        : define                   [forward: full_node]
+// /sessions      : define attach /protocols [forward: full_node]
+// /protocols     : define session

include/bitcoin/node/full_node.hpp

@@ -25,6 +25,8 @@
 #include <bitcoin/node/chasers/chasers.hpp>
 #include <bitcoin/node/configuration.hpp>
 #include <bitcoin/node/define.hpp>
+////#include <bitcoin/node/protocols/protocols.hpp>
+////#include <bitcoin/node/sessions/sessions.hpp>
 
 namespace libbitcoin {
 namespace node {
@@ -40,9 +42,9 @@ class BCN_API full_node
   : public network::net
 {
 public:
-    using memory_controller = block_memory;
     using store = node::store;
     using query = node::query;
+    using memory_controller = block_memory;
     typedef std::shared_ptr<full_node> ptr;
 
     /// Constructors.

include/bitcoin/node/sessions/attach.hpp

@@ -28,7 +28,7 @@
 namespace libbitcoin {
 namespace node {
 
-class session_outbound;
+class full_node;
 
 /// Session base class template for protocol attachment.
 /// node::session does not derive from network::session (siblings).
@@ -40,21 +40,7 @@ class attach
 {
 public:
     attach(full_node& node, uint64_t identifier) NOEXCEPT
-      : Session(node, identifier),
-        session(node),
-        relay_(node.config().network.enable_relay),
-        delay_(node.config().node.delay_inbound),
-        headers_(node.config().node.headers_first),
-        node_network_(to_bool(system::bit_and<uint64_t>
-        (
-            node.config().network.services_maximum,
-            network::messages::peer::service::node_network
-        ))),
-        node_client_filters_(to_bool(system::bit_and<uint64_t>
-        (
-            node.config().network.services_maximum,
-            network::messages::peer::service::node_client_filters
-        )))
+      : Session(node, identifier), session(node)
     {
     }
 
@@ -72,6 +58,20 @@ class attach
 
     void attach_protocols(const network::channel::ptr& channel) NOEXCEPT override
     {
+        const auto relay = config().network.enable_relay;
+        const auto delay = node.config().node.delay_inbound;
+        const auto headers = node.config().node.headers_first;
+        const auto node_network = to_bool(system::bit_and<uint64_t>
+        (
+            node.config().network.services_maximum,
+            network::messages::peer::service::node_network
+        ));
+        const auto node_client_filters = to_bool(system::bit_and<uint64_t>
+        (
+            node.config().network.services_maximum,
+            network::messages::peer::service::node_client_filters
+        ));
+
         using namespace network::messages::peer;
         const auto self = session::shared_from_sibling<attach<Session>,
             network::session>();
@@ -83,7 +83,7 @@ class attach
         channel->attach<protocol_observer>(self)->start();
 
         // Ready to relay blocks or block filters.
-        const auto blocks_out = !delay_ || is_recent();
+        const auto blocks_out = !delay || is_recent();
 
         ///////////////////////////////////////////////////////////////////////
         // bip152: "Upon receipt of a `sendcmpct` message with the first and
@@ -101,12 +101,12 @@ class attach
             channel->attach<protocol_filter_out_70015>(self)->start();
 
         // Node must advertise node_network or no in|out blocks|txs.
-        if (!node_network_)
+        if (!node_network)
             return;
 
         // Ready to relay transactions.
-        const auto txs_in_out = relay_ && peer->is_negotiated(level::bip37) &&
-            (!delay_ || is_current(true));
+        const auto txs_in_out = relay && peer->is_negotiated(level::bip37) &&
+            (!delay || is_current(true));
 
         // Peer advertises chain (blocks in).
         if (peer->is_peer_service(service::node_network))
@@ -164,14 +164,6 @@ class attach
                 network::session::log, socket, node::session::config(),
                 network::session::create_key()));
     }
-
-private:
-    // These are thread safe.
-    const bool relay_;
-    const bool delay_;
-    const bool headers_;
-    const bool node_network_;
-    const bool node_client_filters_;
 };
 
 } // namespace node

include/bitcoin/node/sessions/session.hpp

@@ -21,12 +21,14 @@
 
 #include <bitcoin/database.hpp>
 #include <bitcoin/network.hpp>
+#include <bitcoin/node/configuration.hpp>
 #include <bitcoin/node/define.hpp>
-#include <bitcoin/node/full_node.hpp>
 
 namespace libbitcoin {
 namespace node {
-    
+
+ class full_node;
+
 /// Common session context, presumes will be joined with network::session.
 /// This could be templatized on the sibling, but there only one implemented.
 class BCN_API session
@@ -93,7 +95,7 @@ class BCN_API session
     /// -----------------------------------------------------------------------
 
     /// Thread safe synchronous archival interface.
-    full_node::query& archive() const NOEXCEPT;
+    node::query& archive() const NOEXCEPT;
 
     /// Configuration settings for all libraries.
     const configuration& config() const NOEXCEPT;

src/full_node.cpp

@@ -23,7 +23,8 @@
 #include <bitcoin/network.hpp>
 #include <bitcoin/node/chasers/chasers.hpp>
 #include <bitcoin/node/define.hpp>
-#include <bitcoin/node/sessions/sessions.hpp>
+////#include <bitcoin/node/protocols/protocols.hpp>
+////#include <bitcoin/node/sessions/sessions.hpp>
 
 namespace libbitcoin {
 namespace node {
@@ -419,17 +420,17 @@ network::memory& full_node::get_memory() NOEXCEPT
 
 network::session_manual::ptr full_node::attach_manual_session() NOEXCEPT
 {
-    return net::attach<node::session_manual>(*this);
+    return attach<node::session_manual>(*this);
 }
 
 network::session_inbound::ptr full_node::attach_inbound_session() NOEXCEPT
 {
-    return net::attach<node::session_inbound>(*this);
+    return attach<node::session_inbound>(*this);
 }
 
 network::session_outbound::ptr full_node::attach_outbound_session() NOEXCEPT
 {
-    return net::attach<node::session_outbound>(*this);
+    return attach<node::session_outbound>(*this);
 }
 
 BC_POP_WARNING()