Fix LINKS syntax

to match ircdocs/modern-irc#245
Libera-Chat · Dec 26, 2024 · baed3ef · baed3ef
1 parent 034c4d5
commit baed3ef
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 9 deletions.
diff --git a/sable_ircd/src/command/handlers/links.rs b/sable_ircd/src/command/handlers/links.rs
@@ -4,19 +4,25 @@ use sable_network::prelude::wrapper::ObjectWrapper;
 #[command_handler("LINKS")]
 fn handle_links(
     response: &dyn CommandResponse,
-    server: &ClientServer,
+    local_server: &ClientServer,
     net: &Network,
     source: UserSource,
 ) -> CommandResult {
-    server.policy().can_list_links(&source)?;
+    local_server.policy().can_list_links(&source)?;
 
-    for server in net.servers() {
-        let server_info = format!(
+    for remote_server in net.servers() {
+        let remote_server_info = format!(
             "last_ping={}, version={}",
-            server.last_ping(),
-            server.raw().version
+            remote_server.last_ping(),
+            remote_server.raw().version
         );
-        response.numeric(make_numeric!(Links, server.name(), 0, &server_info));
+        response.numeric(make_numeric!(
+            Links,
+            remote_server.name(),
+            local_server.name(),
+            0,
+            &remote_server_info
+        ));
     }
 
     response.numeric(make_numeric!(EndOfLinks));

diff --git a/sable_ircd/src/messages/numeric.rs b/sable_ircd/src/messages/numeric.rs
@@ -51,8 +51,8 @@ define_messages! {
                                                 => "{chname} {user} {host} * {nick} {status} :{hopcount} {realname}" },
     353(NamesReply)             => { (is_pub: char, chan: &Channel.name(), content: &str)
                                                                 => "{is_pub} {chan} :{content}" },
-    364(Links)                  => { (server_name: &ServerName, hopcount: u64, server_info: &str)
-                                                                => "* {server_name} :{hopcount} {server_info}" },
+    364(Links)                  => { (remote_server_name: &ServerName, local_server_name: &ServerName, hopcount: u64, remote_server_info: &str)
+                                                                => "{remote_server_name} {local_server_name} :{hopcount} {remote_server_info}" },
     365(EndOfLinks)             => { ()                         => "* :End of /LINKS list" },
     366(EndOfNames)             => { (chname: &str)             => "{chname} :End of names list" },