Build conninfo in *Connection.connect()

We'll need to extract 'connect_timeout' from the connection string and
use it outside Connection._connect_gen() in the next commit.

diff --git a/psycopg3/psycopg3/connection.py b/psycopg3/psycopg3/connection.py
index 3e3a90e4a96fea6fa597e411d2c644ac825175a6..22af61d433ed92cc1b6f88b22e3519df951a048d 100644 (file)
--- a/psycopg3/psycopg3/connection.py
+++ b/psycopg3/psycopg3/connection.py
@@ -358,10 +358,8 @@ class BaseConnection(AdaptContext, Generic[Row]):
         *,
         autocommit: bool = False,
         row_factory: Optional[RowFactory[Any]] = None,
-        **kwargs: Any,
     ) -> PQGenConn[ConnectionType]:
         """Generator to connect to the database and create a new instance."""
-        conninfo = make_conninfo(conninfo, **kwargs)
         pgconn = yield from connect(conninfo)
         if not row_factory:
             row_factory = tuple_row
@@ -488,12 +486,10 @@ class Connection(BaseConnection[Row]):
 
         TODO: connection_timeout to be implemented.
         """
+        conninfo = make_conninfo(conninfo, **kwargs)
         return cls._wait_conn(
             cls._connect_gen(
-                conninfo,
-                autocommit=autocommit,
-                row_factory=row_factory,
-                **kwargs,
+                conninfo, autocommit=autocommit, row_factory=row_factory
             )
         )
 
@@ -701,12 +697,10 @@ class AsyncConnection(BaseConnection[Row]):
         row_factory: Optional[RowFactory[Row]] = None,
         **kwargs: Any,
     ) -> "AsyncConnection[Any]":
+        conninfo = make_conninfo(conninfo, **kwargs)
         return await cls._wait_conn(
             cls._connect_gen(
-                conninfo,
-                autocommit=autocommit,
-                row_factory=row_factory,
-                **kwargs,
+                conninfo, autocommit=autocommit, row_factory=row_factory
             )
         )