Fixed bug when setting "SOURCE_ROUTE" on the "DESCRIPTION" section of a

full connect descriptor instead of the "ADDRESS_LIST" section.

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -20,6 +20,8 @@ Thin Mode Changes
 +++++++++++++++++
 
 #)  Fixed bug when using bind variables with scrollable cursors.
+#)  Fixed bug when setting ``SOURCE_ROUTE`` on the ``DESCRIPTION`` section of a
+    full connect descriptor instead of the ``ADDRESS_LIST`` section.
 
 Thick Mode Changes
 ++++++++++++++++++

src/oracledb/impl/base/parsers.pyx

@@ -362,6 +362,7 @@ cdef class ConnectStringParser(BaseParser):
                 description.set_from_security_args(sub_args)
             address_lists = desc_args.get("address_list", desc_args)
             if not isinstance(address_lists, list):
+                description.source_route = False
                 address_lists = [address_lists]
             for list_args in address_lists:
                 address_list = AddressList()

tests/test_4500_connect_params.py

@@ -588,8 +588,7 @@ def test_4531():
         params.parse_connect_string(connect_string)
         source_route_clause = "(SOURCE_ROUTE=ON)" if has_section else ""
         connect_string = (
-            f"(DESCRIPTION={source_route_clause}"
-            f"(ADDRESS_LIST={source_route_clause}"
+            f"(DESCRIPTION=(ADDRESS_LIST={source_route_clause}"
             "(ADDRESS=(PROTOCOL=tcp)(HOST=host1)(PORT=1521))"
             "(ADDRESS=(PROTOCOL=tcp)(HOST=host2)(PORT=1522)))"
             "(CONNECT_DATA=(SERVICE_NAME=my_service_35)))"