make format and introduce new function EnsureRouter on router to check nodedb or do dht lookup

llarp/path/path.cpp

@@ -99,25 +99,28 @@ namespace llarp
     PathContext::ForwardLRCM(const RouterID& nextHop,
                              const std::array< EncryptedFrame, 8 >& frames)
     {
-      auto msg = std::make_shared<const LR_CommitMessage>(frames);
+      auto msg = std::make_shared< const LR_CommitMessage >(frames);
 
       LogDebug("forwarding LRCM to ", nextHop);
       if(m_Router->HasSessionTo(nextHop))
       {
         return m_Router->SendToOrQueue(nextHop, msg.get());
       }
-      const RouterID router = nextHop;
+      const RouterID router   = nextHop;
-      AbstractRouter * const r = m_Router;
+      AbstractRouter* const r = m_Router;
-      m_Router->LookupRouter(nextHop, [msg, r, router](const std::vector<RouterContact> & found) {
+      m_Router->EnsureRouter(
-        if(found.size())
+          nextHop, [msg, r, router](const std::vector< RouterContact >& found) {
-        {
+            if(found.size())
-          r->TryConnectAsync(found[0], 1);
+            {
-          r->SendToOrQueue(router, msg.get());
+              r->TryConnectAsync(found[0], 1);
-        }
+              r->SendToOrQueue(router, msg.get());
-        else
+            }
-          LogError("dropped LRCM to ", router, " as we cannot find in via DHT");
+            else
-      });
+              LogError("dropped LRCM to ", router,
-      LogInfo("we are not directly connected to ", router, " so we need to do a lookup");
+                       " as we cannot find in via DHT");
+          });
+      LogInfo("we are not directly connected to ", router,
+              " so we need to do a lookup");
       return true;
     }
     template < typename Map_t, typename Key_t, typename CheckValue_t,

llarp/router/abstractrouter.cpp

@@ -1,8 +1,19 @@
 #include <router/abstractrouter.hpp>
+#include <nodedb.hpp>
 
 namespace llarp
 {
   AbstractRouter::~AbstractRouter()
   {
   }
+
+  void
+  AbstractRouter::EnsureRouter(RouterID router, RouterLookupHandler handler)
+  {
+    std::vector< RouterContact > found(1);
+    if(nodedb()->Get(router, found[0]))
+      handler(found);
+    else
+      LookupRouter(router, handler);
+  }
 }  // namespace llarp

llarp/router/abstractrouter.hpp

@@ -218,12 +218,16 @@ namespace llarp
     virtual bool
     ConnectionToRouterAllowed(const RouterID &router) const = 0;
 
-    /// return true if we have at least 1 session to this router in either direction
+    /// return true if we have at least 1 session to this router in either
+    /// direction
     virtual bool
-    HasSessionTo(const RouterID & router) const = 0;
+    HasSessionTo(const RouterID &router) const = 0;
 
     virtual util::StatusObject
     ExtractStatus() const = 0;
+
+    void
+    EnsureRouter(RouterID router, RouterLookupHandler handler);
   };
 }  // namespace llarp
 

llarp/router/router.cpp

@@ -360,8 +360,8 @@ namespace llarp
 
     // we don't have the RC locally so do a dht lookup
     _dht->impl->LookupRouter(remote,
-                            std::bind(&Router::HandleDHTLookupForSendTo, this,
+                             std::bind(&Router::HandleDHTLookupForSendTo, this,
-                            remote, std::placeholders::_1));
+                                       remote, std::placeholders::_1));
     return true;
   }
 

llarp/router/router.hpp

@@ -535,8 +535,6 @@ namespace llarp
     bool
     HasSessionTo(const RouterID &remote) const override;
 
-
-
     void
     HandleDHTLookupForTryEstablishTo(
         RouterID remote, const std::vector< RouterContact > &results);

llarp/service/context.cpp

@@ -123,9 +123,9 @@ namespace llarp
         }
       }
       // tick active endpoints
-      for(const auto & item : m_Endpoints)
+      for(const auto &item : m_Endpoints)
       {
-          item.second->Tick(now);
+        item.second->Tick(now);
       }
       /*
       std::vector< RouterID > expired;

llarp/utp/linklayer.cpp

@@ -329,8 +329,7 @@ namespace llarp
     LinkLayer::NewOutboundSession(const RouterContact& rc,
                                   const AddressInfo& addr)
     {
-      return std::make_shared< OutboundSession >(
+      return std::make_shared< OutboundSession >(this, NewSocket(), rc, addr);
-          this, NewSocket(), rc, addr);
     }
 
     uint64