User defined errors raised by the database no longer display an error

help portal URL.

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -20,6 +20,8 @@ Thin Mode Changes
 #)  Added support for writing UTF-8 encoded bytes to CLOB and NCLOB values and
     writing strings to BLOB values in order to be consistent with what is done
     for string variables.
+#)  User defined errors raised by the database no longer display an error help
+    portal URL.
 #)  Fixed potential cursor issues when using DRCP.
 #)  Fixed regression when using IAM token authentication
     (`issue 288 <https://github.com/oracle/python-oracledb/issues/288>`__).

src/oracledb/errors.py

@@ -75,7 +75,11 @@ def _make_adjustments(self):
                 # add Oracle Database Error Help Portal URL for database error
                 # messages, but only in thin mode since this is done
                 # automatically in thick mode with Oracle Client 23c and higher
-                if self.code != 0 and is_thin_mode():
+                if (
+                    self.code != 0
+                    and (self.code < 20000 or self.code >= 21000)
+                    and is_thin_mode()
+                ):
                     self.message = (
                         self.message
                         + "\n"

tests/test_1700_error.py

@@ -182,6 +182,22 @@ def test_1707(self):
         self.assertIsNone(self.cursor.warning)
         self.cursor.execute(f"drop procedure {proc_name}")
 
+    def test_1708(self):
+        "1708 - user defined errors do not generate error help portal URL"
+        for code in (20000, 20500, 20999):
+            with self.assertRaises(oracledb.Error) as cm:
+                self.cursor.execute(
+                    f"""
+                    begin
+                        raise_application_error(-{code}, 'User defined error');
+                    end;
+                    """
+                )
+            error_obj = cm.exception.args[0]
+            self.assertEqual(error_obj.code, code)
+            self.assertEqual(error_obj.full_code, f"ORA-{code}")
+            self.assertTrue("Help:" not in error_obj.message)
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()