Acept string subtypes as connection string

diff --git a/psycopg3/conninfo.py b/psycopg3/conninfo.py
index 0fdce34c4c26234c63311bde0d308bc1e261b99b..f5f07f3616e453f370dd5e5faad34eafa91814c3 100644 (file)
--- a/psycopg3/conninfo.py
+++ b/psycopg3/conninfo.py
@@ -14,10 +14,11 @@ def make_conninfo(conninfo: str = "", **kwargs: Any) -> str:
     if not conninfo and not kwargs:
         return ""
 
-    # If no kwarg is specified don't mung the conninfo but check if it's correct
+    # If no kwarg specified don't mung the conninfo but check if it's correct.
+    # Make sure to return a string, not a subtypep, to avoid making Liskov sad.
     if not kwargs:
         _parse_conninfo(conninfo)
-        return conninfo
+        return str(conninfo)
 
     # Override the conninfo with the parameters
     # Drop the None arguments

diff --git a/tests/test_async_connection.py b/tests/test_async_connection.py
index fa84c2bdd301b9f738c77617ea54f33e1fbff20f..4ea0ee0688158bc78436618ffda3230c245ba205 100644 (file)
--- a/tests/test_async_connection.py
+++ b/tests/test_async_connection.py
@@ -24,6 +24,14 @@ async def test_connect_bad():
         await AsyncConnection.connect("dbname=nosuchdb")
 
 
+async def test_connect_str_subclass(dsn):
+    class MyString(str):
+        pass
+
+    conn = await AsyncConnection.connect(MyString(dsn))
+    assert conn.status == conn.ConnStatus.OK
+
+
 async def test_close(aconn):
     assert not aconn.closed
     await aconn.close()

diff --git a/tests/test_connection.py b/tests/test_connection.py
index 794295b55dd85892b87983f9ecb511284c3a371c..0a83d2f9affda8bab66e050c83780107544ded99 100644 (file)
--- a/tests/test_connection.py
+++ b/tests/test_connection.py
@@ -16,6 +16,14 @@ def test_connect(dsn):
     assert conn.status == conn.ConnStatus.OK
 
 
+def test_connect_str_subclass(dsn):
+    class MyString(str):
+        pass
+
+    conn = Connection.connect(MyString(dsn))
+    assert conn.status == conn.ConnStatus.OK
+
+
 def test_connect_bad():
     with pytest.raises(psycopg3.OperationalError):
         Connection.connect("dbname=nosuchdb")