reject stun urls with transport set

which has been rejected in Chrome since early 2023:
  https://chromiumdash.appspot.com/commits?commit=f4d463ca875f51c26a7de0e594eb1516f3e60111

Author: fippo

src/aiortc/rtcicetransport.py

@@ -10,12 +10,13 @@
 from .exceptions import InvalidStateError
 from .rtcconfiguration import RTCIceServer
 
+# See https://datatracker.ietf.org/doc/html/rfc7064
+# transport is not defined by RFC 7064 and rejected by browsers.
 STUN_REGEX = re.compile(
     r"(?P<scheme>stun|stuns)\:(?P<host>[^?:]+)(\:(?P<port>[0-9]+?))?"
-    # RFC 7064 does not define a "transport" option but some providers
-    # include it, so just ignore it
-    r"(\?transport=.*)?"
+    r"(\?transport=(?P<transport>.*))?"
 )
+# See https://datatracker.ietf.org/doc/html/rfc7065
 TURN_REGEX = re.compile(
     r"(?P<scheme>turn|turns)\:(?P<host>[^?:]+)(\:(?P<port>[0-9]+?))?"
     r"(\?transport=(?P<transport>.*))?"
@@ -162,6 +163,12 @@ def parse_stun_turn_uri(uri: str) -> dict[str, Any]:
         parsed["transport"] = "udp"
     elif parsed["scheme"] == "turns" and not parsed["transport"]:
         parsed["transport"] = "tcp"
+    elif parsed["scheme"] in ["stun", "stuns"]:
+        if parsed["transport"] is not None:
+            raise ValueError(
+                "malformed uri: " + parsed["scheme"] + " must not contain transport"
+            )
+        del parsed["transport"]
 
     return parsed
 

tests/test_rtcicetransport.py

@@ -37,12 +37,11 @@ def test_stun(self) -> None:
             {"stun_server": ("stun.l.google.com", 19302)},
         )
 
-    def test_stun_with_suffix(self) -> None:
+    def test_stun_with_transport(self) -> None:
+        with self.assertRaises(ValueError) as cm:
+            parse_stun_turn_uri("stun:global.stun.twilio.com:3478?transport=udp")
         self.assertEqual(
-            connection_kwargs(
-                [RTCIceServer("stun:global.stun.twilio.com:3478?transport=udp")]
-            ),
-            {"stun_server": ("global.stun.twilio.com", 3478)},
+            str(cm.exception), "malformed uri: stun must not contain transport"
         )
 
     def test_stun_multiple_servers(self) -> None: