Connection parameters that are strings now treat an empty string in the

same way as the value "None".

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -48,6 +48,8 @@ Common Changes
     mode would raise ``DPI-1040: LOB was already closed``.
 #)  Fixed bug in :meth:`ConnectParams.get_connect_string()` when a value for
     the connection parameter ``purity`` has been specified.
+#)  Connection parameters that are strings now treat an empty string in the
+    same way as the value ``None``.
 
 
 oracledb 2.0.1 (January 2024)

src/oracledb/impl/base/utils.pyx

@@ -163,7 +163,7 @@ cdef int _set_str_param(dict args, str name, object target) except -1:
     Python objects).
     """
     in_val = args.get(name)
-    if in_val is not None:
+    if in_val:
         setattr(target, name, str(in_val))
 
 

tests/test_4500_connect_params.py

@@ -215,7 +215,7 @@ def test_4513(self):
         params.parse_connect_string("my_host15:1578/")
         self.assertEqual(params.host, "my_host15")
         self.assertEqual(params.port, 1578)
-        self.assertEqual(params.service_name, "")
+        self.assertEqual(params.service_name, None)
 
     def test_4514(self):
         "4514 - test missing entry in tnsnames"
@@ -769,6 +769,13 @@ def test_4566(self):
         params.set(sdu=2097153)
         self.assertEqual(params.sdu, 2097152)
 
+    def test_4567(self):
+        "4567 - test empty connection class"
+        params = oracledb.ConnectParams()
+        self.assertEqual(params.cclass, None)
+        params.set(cclass="")
+        self.assertEqual(params.cclass, None)
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()