From 2347f90bda865d954298d0f1cd7c7626157339cd Mon Sep 17 00:00:00 2001
From: Kylie Smith <kylie.smithy01@gmail.com>
Date: Mon, 8 Apr 2024 11:39:40 +1000
Subject: [PATCH] Split client channels into hi and lo for JS

---
 src/clientagent/client_agent.cpp | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/clientagent/client_agent.cpp b/src/clientagent/client_agent.cpp
index 5726680..0cc5dee 100644
--- a/src/clientagent/client_agent.cpp
+++ b/src/clientagent/client_agent.cpp
@@ -362,7 +362,10 @@ void ClientAgent::HandleWeb(ws28::Client *client, nlohmann::json &data) {
     nlohmann::json clientInfo = nlohmann::json::array();
     for (const auto &participant : _participants) {
       clientInfo.push_back({
-          {"channel", participant->GetChannel()},
+          // We have to do this terribleness because JavaScript doesn't support
+          // uint64's (see handling 'client' messages below.)
+          {"channelHi", (participant->GetChannel() >> 32) & 0xFFFFFFFF},
+          {"channelLo", participant->GetChannel() & 0xFFFFFFFF},
           {"ip", participant->GetRemoteAddress().ip},
           {"port", participant->GetRemoteAddress().port},
           {"state", participant->GetAuthState()},
@@ -384,7 +387,10 @@ void ClientAgent::HandleWeb(ws28::Client *client, nlohmann::json &data) {
                                {"clients", clientInfo},
                            });
   } else if (data["msg"] == "client") {
-    auto channel = data["channel"].template get<uint64_t>();
+    // We have to do this terribleness because JavaScript doesn't support
+    // uint64's.
+    auto channel = (uint64_t)data["channelHi"].template get<uint32_t>() << 32 |
+                   data["channelLo"].template get<uint32_t>();
 
     // Try to find a matching client for the provided channel.
     auto participant =
@@ -418,7 +424,7 @@ void ClientAgent::HandleWeb(ws28::Client *client, nlohmann::json &data) {
     // Build an active interests array.
     nlohmann::json interests = nlohmann::json::array();
     for (const auto &interest : (*participant)->GetInterests()) {
-      ownedObjs.push_back({{"id", interest.first},
+      interests.push_back({{"id", interest.first},
                            {"parent", interest.second.parent},
                            {"zones", interest.second.zones}});
     }