gitpython-developers
diff --git a/‎git/util.py‎
Lines changed: 4 additions & 0 deletions b/‎git/util.py‎
Lines changed: 4 additions & 0 deletions
diff --git a/‎test/test_actor.py‎
Lines changed: 9 additions & 2 deletions b/‎test/test_actor.py‎
Lines changed: 9 additions & 2 deletions
@@ -858,6 +858,10 @@ class Actor:
     committers and authors or anything with a name and an email as mentioned in the git
     log entries."""
 
+    # Kept for compatibility only: searching long malformed strings with this regex can
+    # take quadratic time. ``_from_string`` deliberately uses delimiter scans instead.
+    name_email_regex = re.compile(r"(.*) <(.*?)>")
+
     # ENVIRONMENT VARIABLES
     # These are read when creating new commits.
     env_author_name = "GIT_AUTHOR_NAME"
 
@@ -3,6 +3,8 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
+from unittest import mock
+
 from git import Actor
 
 from test.lib import TestBase
@@ -29,10 +31,15 @@ def test_from_string_should_handle_just_name(self):
 
     def test_from_string_handles_unterminated_email_without_regex_backtracking(self):
         value = "A" * 20_000 + " <unterminated"
-        actor = Actor._from_string(value)
-        self.assertNotIn("name_email_regex", vars(Actor))
+        with mock.patch.object(Actor, "name_email_regex", None):
+            actor = Actor._from_string(value)
         self.assertEqual(actor, Actor(value, None))
 
+    def test_name_email_regex_is_available(self):
+        match = Actor.name_email_regex.match("Michael Trier <mtrier@example.com>")
+        self.assertIsNotNone(match)
+        self.assertEqual(match.groups(), ("Michael Trier", "mtrier@example.com"))
+
     def test_from_string_does_not_parse_across_lines(self):
         self.assertEqual(Actor._from_string("x <a>\n y <b>"), Actor("x", "a"))