fix(utils): treat empty-string pattern like empty list in match_regex_list

Address review feedback: an empty string in regex_list represents a
typo/mistake and should be treated as if [] was passed, returning False
rather than matching every item. Fold the regression case into the
parameterized test_match_regex_list cases and drop the redundant
standalone test.

Author: devteamaegis

sentry_sdk/utils.py

@@ -1740,7 +1740,10 @@ def match_regex_list(
         return False
 
     for item_matcher in regex_list:
-        if not substring_matching and (not item_matcher or item_matcher[-1] != "$"):
+        if not item_matcher:
+            return False
+
+        if not substring_matching and item_matcher[-1] != "$":
             item_matcher += "$"
 
         matched = re.search(item_matcher, item)

tests/test_utils.py

@@ -556,20 +556,13 @@ def test_include_source_context_when_serializing_frame(include_source_context):
         ["some-string", ["some.*"], True],
         ["some-string", ["Some"], False],  # we do case sensitive matching
         ["some-string", [".*string$"], True],
+        ["some-string", [""], False],  # an empty-string pattern is treated like []
     ],
 )
 def test_match_regex_list(item, regex_list, expected_result):
     assert match_regex_list(item, regex_list) == expected_result
 
 
-def test_match_regex_list_empty_string_pattern():
-    # An empty-string pattern must not raise IndexError (regression test).
-    result = match_regex_list("anything", [""])
-    assert isinstance(result, bool)
-    assert match_regex_list("foobar", ["foo"]) is False
-    assert match_regex_list("foo", ["foo"]) is True
-
-
 @pytest.mark.parametrize(
     "version,expected_result",
     [