Skip to content

Commit de1b57a

Browse files
Fix possible regex timeout
1 parent 6e1cf58 commit de1b57a

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

src/App.cs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,12 @@ public async Task Run(Settings settings)
2727
var labels = await _sync.GetAccountLabels(account);
2828
_log(string.Empty);
2929

30-
var filteredRepos = settings.Filters.Any()
31-
? allRepos.Where(r => settings.Filters.Any(f => new Regex(f).IsMatch(r.Name)))
30+
var regexFilters = settings.Filters
31+
.Select(f => new Regex(f, RegexOptions.None, TimeSpan.FromSeconds(1)))
32+
.ToArray();
33+
34+
var filteredRepos = regexFilters.Any()
35+
? allRepos.Where(r => regexFilters.Any(f => f.IsMatch(r.Name)))
3236
: allRepos;
3337

3438
var repos = filteredRepos.ToArray();

src/GitHubLabelSync.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22
<PropertyGroup>
3-
<Version>2.1.0</Version>
3+
<Version>2.1.1</Version>
44
<OutputType>Exe</OutputType>
55
<PackAsTool>true</PackAsTool>
66
<ToolCommandName>sync-labels</ToolCommandName>

0 commit comments

Comments
 (0)