Fixed bug when calling Connection.gettype() for a type that exists but

on which the user has insufficient privileges to view (#397).

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -31,6 +31,9 @@ Thin Mode Changes
 #)  Added :meth:`oracledb.register_protocol()` to allow users to register a
     function that will be called when a particular protocol is detected in a
     connection string.
+#)  Fixed bug when calling :meth:`Connection.gettype()` for a type that exists
+    but on which the user has insufficient privileges to view
+    (`issue 397 <https://github.com/oracle/python-oracledb/issues/397>`__).
 #)  Fixed bug when calling :meth:`ConnectParams.parse_dsn_with_credentials()`
     with an Easy Connect string containing a protocol.
 #)  Fixed bug when calling :meth:`Cursor.parse()` with autocommit enabled.

src/oracledb/impl/thin/dbobject_cache.pyx

@@ -416,6 +416,9 @@ cdef class BaseThinDbObjectTypeCache:
             typ_impl.schema = self.schema_var.getvalue()
             typ_impl.package_name = self.package_name_var.getvalue()
             typ_impl.name = self.name_var.getvalue()
+            if typ_impl.name is None:
+                errors._raise_err(errors.ERR_INVALID_OBJECT_TYPE_NAME,
+                                  name=name)
             typ_impl.is_xml_type = \
                     (typ_impl.schema == "SYS" and typ_impl.name == "XMLTYPE")
         typ_impl.attrs = []

tests/test_2300_object_var.py

@@ -811,6 +811,15 @@ def test_2340(self):
         var.setvalue(0, obj)
         self.assertIs(var.getvalue(), obj)
 
+    def test_2341(self):
+        "2341 - test insufficient privileges for gettype()"
+        user = test_env.get_proxy_user()
+        password = test_env.get_proxy_password()
+        main_user = test_env.get_main_user().upper()
+        conn = test_env.get_connection(user=user, password=password)
+        with self.assertRaisesFullCode("DPY-2035"):
+            conn.gettype(f"{main_user}.UDT_OBJECTARRAY")
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()

tests/test_5600_dbobject_async.py

@@ -589,6 +589,17 @@ async def test_5617(self):
         result = [i for i in obj]
         self.assertEqual(result, [5, 10, 15])
 
+    async def test_5618(self):
+        "5618 - test insufficient privileges for gettype()"
+        user = test_env.get_proxy_user()
+        password = test_env.get_proxy_password()
+        main_user = test_env.get_main_user().upper()
+        async with await test_env.get_connection_async(
+            user=user, password=password
+        ) as conn:
+            with self.assertRaisesFullCode("DPY-2035"):
+                await conn.gettype(f"{main_user}.UDT_OBJECTARRAY")
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()