From 31529635452dc5742dfbc7502cfbb4a1283ca911 Mon Sep 17 00:00:00 2001
From: Dominic Farolino <domfarolino@gmail.com>
Date: Thu, 24 Aug 2023 09:27:32 -0400
Subject: [PATCH] Remove `Channel::remote_node_name_`

---
 mage/core/channel.cc | 10 +++-------
 mage/core/channel.h  |  5 +----
 mage/core/node.cc    |  2 +-
 3 files changed, 5 insertions(+), 12 deletions(-)

diff --git a/mage/core/channel.cc b/mage/core/channel.cc
index 4fbce79..3d055f4 100644
--- a/mage/core/channel.cc
+++ b/mage/core/channel.cc
@@ -74,12 +74,8 @@ void Channel::Start() {
   io_task_loop_.WatchSocket(this);
 }
 
-void Channel::SetRemoteNodeName(const std::string& name) {
-  CHECK_ON_THREAD(base::ThreadType::UI);
-  remote_node_name_ = name;
-}
-
 void Channel::SendInvitation(std::string inviter_name,
+                             std::string temporary_remote_node_name,
                              std::string intended_endpoint_peer_name) {
   CHECK_ON_THREAD(base::ThreadType::UI);
   Message message(MessageType::SEND_INVITATION);
@@ -91,8 +87,8 @@ void Channel::SendInvitation(std::string inviter_name,
          inviter_name.size());
 
   // Serialize temporary remote node name.
-  memcpy(params.data()->temporary_remote_node_name, remote_node_name_.c_str(),
-         remote_node_name_.size());
+  memcpy(params.data()->temporary_remote_node_name,
+         temporary_remote_node_name.c_str(), temporary_remote_node_name.size());
 
   // Serialize intended endpoint peer name.
   memcpy(params.data()->intended_endpoint_peer_name,
diff --git a/mage/core/channel.h b/mage/core/channel.h
index 59b7bb6..5e07f5e 100644
--- a/mage/core/channel.h
+++ b/mage/core/channel.h
@@ -25,6 +25,7 @@ class Channel : public base::TaskLoopForIO::SocketReader {
   void Start();
   void SetRemoteNodeName(const std::string& name);
   void SendInvitation(std::string inviter_name,
+                      std::string temporary_remote_node_name,
                       std::string intended_endpoint_peer_name);
   void SendAcceptInvitation(std::string temporary_remote_node_name,
                             std::string actual_node_name,
@@ -35,10 +36,6 @@ class Channel : public base::TaskLoopForIO::SocketReader {
   void OnCanReadFromSocket() override;
 
  private:
-  // This is always initialized as something temporary until we hear back from
-  // the remote node and it tells us its name.
-  std::string remote_node_name_;
-
   // The |Delegate| owns |this|, so this pointer will never be null.
   Delegate* delegate_;
 
diff --git a/mage/core/node.cc b/mage/core/node.cc
index 1e6c60d..bdb450c 100644
--- a/mage/core/node.cc
+++ b/mage/core/node.cc
@@ -123,8 +123,8 @@ MessagePipe Node::SendInvitationAndGetMessagePipe(int fd) {
   // expensive, and we can avoid it with one-time proper ordering.
   std::unique_ptr<Channel>& channel = it.first->second;
   channel->Start();
-  channel->SetRemoteNodeName(temporary_remote_node_name);
   channel->SendInvitation(/*inviter_name=*/name_,
+                          /*temporary_remote_node_name=*/temporary_remote_node_name,
                           /*intended_endpoint_peer_name=*/
                           remote_endpoint->peer_address.endpoint_name);