twilio-labs / twilio-aspnet

Integrate Twilio Programmable Messaging and Voice with ASP.NET Respond to webhooks with TwiML in seconds
Apache License 2.0
59 stars 30 forks source link

RequestValidationHelper Ignores allowLocal #9

Closed tofer closed 6 years ago

tofer commented 6 years ago

The Twilio.AspNet.Core.RequestValidationHelper class ignores the allowLocal parameter on one of the overloads.

Currently:

public bool IsValidRequest(HttpContext context, string authToken, bool allowLocal = true)
{
    return IsValidRequest(context, authToken, null);
}

Should be:

public bool IsValidRequest(HttpContext context, string authToken, bool allowLocal = true)
{
    return IsValidRequest(context, authToken, null, allowLocal);
}
tofer commented 6 years ago

Just realized you accept pull requests, which I just submitted. Cheers.

dprothero commented 6 years ago

Thanks! Sorry for the delay on this.