Pilz.NET/Pilz
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

improve TryGetHandler to find direct matches before checking via RegEx

Browse Source
This commit is contained in:
Pilzinsel64
2025-01-28 08:25:26 +01:00
parent 1b1f92dea5
commit 5d941506f6
1 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
Pilz.Net/Api/ApiServer.cs
View File

@@ -408,16 +408,14 @@ public class ApiServer(string apiUrl) : IApiServer
protected virtual bool TryGetHandler(string url, string? query, string method, [NotNullWhen(true)] out PrivateMessageHandler? handler) protected virtual bool TryGetHandler(string url, string? query, string method, [NotNullWhen(true)] out PrivateMessageHandler? handler)
{ {
handler = handlers.FirstOrDefault(handler => // Filter by method
{ var filtered = handlers.Where(handler => handler.Attribute.Methods.Contains(method));
if (!handler.Attribute.Methods.Contains(method))
return false;
if (handler.UseRegEx) // Check if equals via string comparation (ignore-case)
return Regex.IsMatch(url, handler.Url, RegexOptions.IgnoreCase); handler = filtered.FirstOrDefault(handler => handler.Url.Equals(url, StringComparison.InvariantCultureIgnoreCase));
return handler.Url.Equals(url, StringComparison.InvariantCultureIgnoreCase); // Check if equals via RegEx
}); handler ??= filtered.Where(n => n.UseRegEx).FirstOrDefault(handler => handler.Url.Equals(url, StringComparison.InvariantCultureIgnoreCase));
return handler != null; return handler != null;
} }