misc: update webhook regex to support discord.com (#1551)
* Updating webhook regex for discord.com Updates webhook URL regex matching for discordapp.com and discord.com * Fixing comment * Whitespace
This commit is contained in:
@@ -33,7 +33,7 @@ namespace Discord.Webhook
|
||||
: this(webhookUrl, new DiscordRestConfig()) { }
|
||||
|
||||
// regex pattern to match webhook urls
|
||||
private static Regex WebhookUrlRegex = new Regex(@"^.*discordapp\.com\/api\/webhooks\/([\d]+)\/([a-z0-9_-]+)$", RegexOptions.Compiled | RegexOptions.IgnoreCase);
|
||||
private static Regex WebhookUrlRegex = new Regex(@"^.*(discord|discordapp)\.com\/api\/webhooks\/([\d]+)\/([a-z0-9_-]+)$", RegexOptions.Compiled | RegexOptions.IgnoreCase);
|
||||
|
||||
/// <summary> Creates a new Webhook Discord client. </summary>
|
||||
public DiscordWebhookClient(ulong webhookId, string webhookToken, DiscordRestConfig config)
|
||||
@@ -132,13 +132,13 @@ namespace Discord.Webhook
|
||||
if (match != null)
|
||||
{
|
||||
// ensure that the first group is a ulong, set the _webhookId
|
||||
// 0th group is always the entire match, so start at index 1
|
||||
if (!(match.Groups[1].Success && ulong.TryParse(match.Groups[1].Value, NumberStyles.None, CultureInfo.InvariantCulture, out webhookId)))
|
||||
// 0th group is always the entire match, and 1 is the domain; so start at index 2
|
||||
if (!(match.Groups[2].Success && ulong.TryParse(match.Groups[2].Value, NumberStyles.None, CultureInfo.InvariantCulture, out webhookId)))
|
||||
throw ex("The webhook Id could not be parsed.");
|
||||
|
||||
if (!match.Groups[2].Success)
|
||||
if (!match.Groups[3].Success)
|
||||
throw ex("The webhook token could not be parsed.");
|
||||
webhookToken = match.Groups[2].Value;
|
||||
webhookToken = match.Groups[3].Value;
|
||||
}
|
||||
else
|
||||
throw ex();
|
||||
|
||||
Reference in New Issue
Block a user