Document reflector destination call

[m17rt] / tools / m17rt-netclient / src / main.rs

diff --git a/tools/m17rt-netclient/src/main.rs b/tools/m17rt-netclient/src/main.rs
index fbcfb155284981e5f958355c97937a1797fca35d..5fee7d6ed5275296b66a4c3f02216cdc7c5b0e7a 100644 (file)
--- a/tools/m17rt-netclient/src/main.rs
+++ b/tools/m17rt-netclient/src/main.rs
@@ -34,6 +34,14 @@ fn main() {
                 .required(true)
                 .help("Your callsign for reflector registration and transmissions"),
         )
                 .required(true)
                 .help("Your callsign for reflector registration and transmissions"),
         )

+        .arg(
+            Arg::new("reflector")
+                .long("reflector")
+                .short('r')
+                .value_parser(valid_callsign)
+                .required(true)
+                .help("Reflector designator/callsign, often starting with 'M17-'"),
+        )

         .arg(
             Arg::new("module")
                 .long("module")
         .arg(
             Arg::new("module")
                 .long("module")


@@ -59,11 +67,24 @@ fn main() {
     let hostname = args.get_one::<String>("hostname").unwrap();
     let port = args.get_one::<u16>("port").unwrap();
     let callsign = args.get_one::<M17Address>("callsign").unwrap();
     let hostname = args.get_one::<String>("hostname").unwrap();
     let port = args.get_one::<u16>("port").unwrap();
     let callsign = args.get_one::<M17Address>("callsign").unwrap();

+    let reflector = args.get_one::<M17Address>("reflector").unwrap();

     let module = args.get_one::<char>("module").unwrap();
     let input = args.get_one::<String>("input");
     let output = args.get_one::<String>("output");
 
     let module = args.get_one::<char>("module").unwrap();
     let input = args.get_one::<String>("input");
     let output = args.get_one::<String>("output");
 

-    let mut tx = Codec2TxAdapter::new(callsign.clone(), M17Address::new_broadcast());
+    // It is current convention that mrefd requires the destination of transmissions to match the reflector.
+    // If you are connected to "M17-XXX" on module B then you must set the dst to "M17-XXX B".
+    // This requirement is likely to change but for the purposes of this test client we'll hard-code the
+    // behaviour for the time being.
+    let ref_with_mod = format!("{} {}", reflector, module);
+    let Ok(reflector) = M17Address::from_callsign(&ref_with_mod) else {
+        println!(
+            "Unable to create valid destination address for reflector + callsign '{ref_with_mod}'"
+        );
+        std::process::exit(1);
+    };
+
+    let mut tx = Codec2TxAdapter::new(callsign.clone(), reflector);

     if let Some(input) = input {
         tx.set_input_card(input);
     }
     if let Some(input) = input {
         tx.set_input_card(input);
     }