liz/PluralKit
liz/PluralKit
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: check if member target is 5-length alpha before trying to search by HID (#441)

Browse Source
This commit is contained in:
spiral 2022-04-24 14:51:47 -04:00
parent 183779e055
commit bc2cfbeba3
No known key found for this signature in database
GPG Key ID: 244A11E4B0BCF40E
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
PluralKit.Bot/CommandSystem/Context/ContextEntityArgumentsExt.cs
View File

@ -1,3 +1,5 @@
using System.Text.RegularExpressions;
using Myriad.Extensions; using Myriad.Extensions;
using Myriad.Types; using Myriad.Types;
@ -79,7 +81,10 @@ public static class ContextEntityArgumentsExt
return memberByDisplayName; return memberByDisplayName;
} }
// Finally (or if by-HID lookup is specified), try member HID parsing: // Finally (or if by-HID lookup is specified), check if input is a valid HID and then try member HID parsing:
if (!Regex.IsMatch(input, @"^[a-zA-Z]{5}$"))
return null;
// For posterity: // For posterity:
// There was a bug that made `SELECT * FROM MEMBERS WHERE HID = $1` hang forever BUT // There was a bug that made `SELECT * FROM MEMBERS WHERE HID = $1` hang forever BUT