Fix possible regex timeout

ecoAPM · Mar 24, 2023 · de1b57a · de1b57a
1 parent 6e1cf58
commit de1b57a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/App.cs b/src/App.cs
@@ -27,8 +27,12 @@ public async Task Run(Settings settings)
 		var labels = await _sync.GetAccountLabels(account);
 		_log(string.Empty);
 
-		var filteredRepos = settings.Filters.Any()
-			? allRepos.Where(r => settings.Filters.Any(f => new Regex(f).IsMatch(r.Name)))
+		var regexFilters = settings.Filters
+			.Select(f => new Regex(f, RegexOptions.None, TimeSpan.FromSeconds(1)))
+			.ToArray();
+
+		var filteredRepos = regexFilters.Any()
+			? allRepos.Where(r => regexFilters.Any(f => f.IsMatch(r.Name)))
 			: allRepos;
 
 		var repos = filteredRepos.ToArray();

diff --git a/src/GitHubLabelSync.csproj b/src/GitHubLabelSync.csproj
@@ -1,6 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
-		<Version>2.1.0</Version>
+		<Version>2.1.1</Version>
 		<OutputType>Exe</OutputType>
 		<PackAsTool>true</PackAsTool>
 		<ToolCommandName>sync-labels</ToolCommandName>