fixup! Adjust connect arguments to enable SSL using SQLAlchemy DB URI

Author: amotl

CHANGES.txt

@@ -13,7 +13,8 @@ Unreleased
 
 - Adjusted connect arguments to accept credentials within the HTTP URI.
 
-- Optionally enable SSL using SQLAlchemy DB URI with parameter ``?ssl=true``.
+- Added support for enabling SSL using SQLAlchemy DB URI with parameter
+  ``?ssl=true``.
 
 2020/09/28 0.26.0
 =================

src/crate/client/sqlalchemy/dialect.py

@@ -197,10 +197,9 @@ def connect(self, host=None, port=None, *args, **kwargs):
             server = kwargs.pop('servers')
         servers = to_list(server)
         if servers:
-            if "ssl" in kwargs:
-                use_ssl = asbool(kwargs.pop("ssl"))
-                if use_ssl:
-                    servers = ["https://" + server for server in servers]
+            use_ssl = asbool(kwargs.pop("ssl", False))
+            if use_ssl:
+                servers = ["https://" + server for server in servers]
             return self.dbapi.connect(servers=servers, **kwargs)
         return self.dbapi.connect(**kwargs)