Merge pull request #21014 from hvitved/csharp/file-path-filter-log-levels

hvitved · web-flow · commit 6feded900059 · 2025-12-11T14:02:51.000+01:00
C#: Adjust log levels in `FilePathFilter.cs`
diff --git a/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/FilePathFilter.cs b/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/FilePathFilter.cs
@@ -55,12 +55,12 @@ public IEnumerable<FileInfo> Filter(IEnumerable<FileInfo> files)
                 }
                 else
                 {
-                    logger.LogInfo($"Invalid filter: {filter}");
+                    logger.LogWarning($"Invalid filter: {filter}");
                     continue;
                 }
 
                 var regex = new FilePattern(filterText).RegexPattern;
-                logger.LogInfo($"Filtering {(include ? "in" : "out")} files matching '{regex}'. Original glob filter: '{filter}'");
+                logger.LogDebug($"Filtering {(include ? "in" : "out")} files matching '{regex}'. Original glob filter: '{filter}'");
                 pathFilters.Add(new PathFilter(new Regex(regex, RegexOptions.IgnoreCase | RegexOptions.Compiled | RegexOptions.Singleline), include));
             }
 
@@ -91,7 +91,7 @@ public IEnumerable<FileInfo> Filter(IEnumerable<FileInfo> files)
 
                 if (!include)
                 {
-                    logger.LogInfo($"Excluding '{f.FileInfo.FullName}'");
+                    logger.LogDebug($"Excluding '{f.FileInfo.FullName}'");
                 }
 
                 return include;

Original file line number	Diff line number	Diff line change
`@@ -55,12 +55,12 @@ public IEnumerable<FileInfo> Filter(IEnumerable<FileInfo> files)`
`55`	`55`	`}`
`56`	`56`	`else`
`57`	`57`	`{`
`58`		`- logger.LogInfo($"Invalid filter: {filter}");`
	`58`	`+ logger.LogWarning($"Invalid filter: {filter}");`
`59`	`59`	`continue;`
`60`	`60`	`}`
`61`	`61`
`62`	`62`	`var regex = new FilePattern(filterText).RegexPattern;`
`63`		`- logger.LogInfo($"Filtering {(include ? "in" : "out")} files matching '{regex}'. Original glob filter: '{filter}'");`
	`63`	`+ logger.LogDebug($"Filtering {(include ? "in" : "out")} files matching '{regex}'. Original glob filter: '{filter}'");`
`64`	`64`	`pathFilters.Add(new PathFilter(new Regex(regex, RegexOptions.IgnoreCase \| RegexOptions.Compiled \| RegexOptions.Singleline), include));`
`65`	`65`	`}`
`66`	`66`
`@@ -91,7 +91,7 @@ public IEnumerable<FileInfo> Filter(IEnumerable<FileInfo> files)`
`91`	`91`
`92`	`92`	`if (!include)`
`93`	`93`	`{`
`94`		`- logger.LogInfo($"Excluding '{f.FileInfo.FullName}'");`
	`94`	`+ logger.LogDebug($"Excluding '{f.FileInfo.FullName}'");`
`95`	`95`	`}`
`96`	`96`
`97`	`97`	`return include;`