Remove support for tcp:// and socket:// DSN schemes

changelog.md

@@ -1,6 +1,11 @@
 Upcoming (TBD)
 ==============
 
+Breaking Changes
+---------
+* Remove undocumented support for `tcp://` and `socket://` DSNs.
+
+
 Documentation
 ---------
 * Fix a typo in myclirc commentary.

mycli/packages/cli_utils.py

@@ -23,11 +23,7 @@ def is_valid_connection_scheme(text: str) -> tuple[bool, str | None]:
     # exit early if the text does not resemble a DSN URI
     if "://" not in text:
         return False, None
-    scheme = text.split("://")[0]
-    if scheme.startswith('mysql+'):
-        return True, None
-    if scheme.startswith('mysqlx+'):
-        return True, None
-    if scheme in ("mysql", "mysqlx", "tcp", "socket"):
+    scheme = text.split("://")[0].split("+")[0]
+    if scheme in ("mysql", "mysqlx"):
         return True, None
     return False, scheme

test/pytests/test_cli_utils.py

@@ -38,8 +38,8 @@ def test_filtered_sys_argv_appends_empty_password_sentinel(monkeypatch, password
         ('mysql+pymysql://user@localhost/db', True, None),
         ('mysqlx://user@localhost/db', True, None),
         ('mysqlx+pymysql://user@localhost/db', True, None),
-        ('tcp://localhost:3306', True, None),
-        ('socket:///tmp/mysql.sock', True, None),
+        ('tcp://localhost:3306', False, 'tcp'),
+        ('socket:///tmp/mysql.sock', False, 'socket'),
         ('ssh://user@example.com', False, 'ssh'),
         ('postgres://user@localhost/db', False, 'postgres'),
         ('http://example.com', False, 'http'),