Fix regex pattern to match exception names with alphanumeric characters and underscores in sentry_tools.py

The regular expression pattern used to find exceptions in strings has been updated to include alphanumeric characters and underscores. This ensures that exceptions with names like "pony_orm.core.TransactionIntegrityError" are correctly matched.

Also, added test cases for strings that should not be matched in test_sentry_tools.py.

Author: drew2a

src/tribler/core/sentry_reporter/sentry_tools.py

@@ -11,7 +11,7 @@
 CONTEXT_DELIMITER = '--CONTEXT--'
 
 # Find an exception in the string like: "OverflowError: bind(): port must be 0-65535"
-_re_search_exception = re.compile(r'^(\S+):\s+(.+)')
+_re_search_exception = re.compile(r'^([A-Za-z0-9_.]+):\s+(.+)')
 _re_search_exception_exclusions = re.compile(r'(?:warning)', re.RegexFlag.IGNORECASE)
 
 # Remove the substring like "Sentry is attempting to send 1 pending error messages"

src/tribler/core/sentry_reporter/tests/test_sentry_tools.py

@@ -151,16 +151,27 @@ def test_extract_dict():
 ]
 
 EXCEPTION_STRINGS = [
-    ('OverflowError: bind(): port must be 0-65535',
-     ('OverflowError', 'bind(): port must be 0-65535'),
-     ),
-
-    ("pony.orm.core.TransactionIntegrityError: MiscData['db_version'] cannot be stored. IntegrityError: UNIQUE",
-     ('pony.orm.core.TransactionIntegrityError', "MiscData['db_version'] cannot be stored. IntegrityError: UNIQUE"),
-     ),
-
+    (
+        'OverflowError: bind(): port must be 0-65535',
+        (
+            'OverflowError',
+            'bind(): port must be 0-65535'
+        ),
+    ),
+
+    (
+        "pony_orm.core.TransactionIntegrityError: MiscData['db_version'] cannot be stored. IntegrityError: UNIQUE",
+        (
+            'pony_orm.core.TransactionIntegrityError',
+            "MiscData['db_version'] cannot be stored. IntegrityError: UNIQUE"
+        ),
+    ),
+
+    # Strings thst should not be matched
     ('ERROR <exception_handler:100>', None),
     ('PyInstaller\loader\pyimod03_importers.py:495', None),
+    ('foo:bar: baz', None),
+    ('foo<bar>: baz', None),
 ]