fix parsing of USA callsigns--specifically different prefix/suffix counts like 1x2 and 1x3 calls

Author: NeilHanlon

apprise/utils/parse.py

@@ -107,18 +107,18 @@
     re.I,
 )
 
-# A simple verification check to make sure the content specified
-# rougly conforms to a ham radio call sign before we parse it further
+# Quick sanity check - does this look like a valid callsign before we
+# bother parsing it
 IS_CALL_SIGN = re.compile(
-    r"^(?P<callsign>[a-z0-9]{2,3}[0-9][a-z0-9]{3})"
-    r"(?P<ssid>-[a-z0-9]{1,2})?\s*$",
+    r"^(?P<callsign>[a-z]{1,3}[0-9][a-z]{1,3})"
+    r"(?P<ssid>-(?:[0-9]|1[0-5]))?\s*$",
     re.I,
 )
 
-# Regular expression used to destinguish between multiple ham radio call signs
+# Regex to split multiple callsigns from a single string
 CALL_SIGN_DETECTION_RE = re.compile(
-    r"\s*([a-z0-9]{2,3}[0-9][a-z0-9]{3}(?:-[a-z0-9]{1,2})?)"
-    r"(?=$|[\s,]+[a-z0-9]{4,6})",
+    r"\s*([a-z0-9]{1,2}[0-9][a-z0-9]{1,3}(?:-(?:[0-9]|1[0-5]))?)"
+    r"(?=\s*$|\s*[,;\s]+)",
     re.I,
 )
 

tests/test_apprise_utils.py

@@ -1685,14 +1685,38 @@ def test_is_call_sign_no():
     assert utils.parse.is_call_sign(42) is False
 
     # To short or 2 long
-    assert utils.parse.is_call_sign("DF1AB") is False
     assert utils.parse.is_call_sign("DF1ABCX") is False
     assert utils.parse.is_call_sign("DF1ABCEFG") is False
     assert utils.parse.is_call_sign("1ABCX") is False
     # 4th character is not an number
     assert utils.parse.is_call_sign("XXXXXX") is False
 
     # Some valid checks
+    # 1x2
+    result = utils.parse.is_call_sign("A0AF")
+    assert isinstance(result, dict)
+    assert result["callsign"] == "A0AF"
+    assert result["ssid"] == ""
+
+    # 2x1
+    result = utils.parse.is_call_sign("AA0A")
+    assert isinstance(result, dict)
+    assert result["callsign"] == "AA0A"
+    assert result["ssid"] == ""
+
+    # 2x2
+    result = utils.parse.is_call_sign("AA0AF")
+    assert isinstance(result, dict)
+    assert result["callsign"] == "AA0AF"
+    assert result["ssid"] == ""
+
+    # 1x3
+    result = utils.parse.is_call_sign("K0ACL")
+    assert isinstance(result, dict)
+    assert result["callsign"] == "K0ACL"
+    assert result["ssid"] == ""
+
+    # 2x3
     result = utils.parse.is_call_sign("DF1ABC")
     assert isinstance(result, dict)
     assert result["callsign"] == "DF1ABC"

tests/test_plugin_aprs.py

@@ -80,6 +80,12 @@ def test_plugin_aprs_urls(mock_create_connection):
     )
     assert instance.notify("test") is True
 
+    # 1N3 callsigns
+    instance = apprise.Apprise.instantiate(
+        "aprs://D1JSL-15:12345@D1ABC"
+    )
+    assert isinstance(instance, NotifyAprs)
+
     instance = apprise.Apprise.instantiate(
         "aprs://DF1JSL-15:12345@DF1ABC?delay=3.0"
     )