Address some PR comments from #153

[dougbu]

• more for #5
• grab some fixes from aspnet/AspNetWebStack repo
• revamp IConfiguration use
  • remove IWebHookReceiverConfig, its implementation and extension methods
  • make methods synchronous if they were async only for configuration lookups
  • remove configuration value caching in PusherVerifySignatureFilter
• remove HttpRequest.IsLocal() extension method
  • instead check for development environment
• move JSON and XML dependencies into specific receiver projects
• make default WebHookAttribute constructor internal and move its doc comments to GeneralWebHookAttribute
• add WebHookConstants.EventKeyNames
  • add RouteData.TryGetEventName(...) extension method; WebHookPingResponseFilter does not need full array

Note this PR does not close questions @rynowak and I had left unfinished in #153.

[dnfclas]

@dougbu, Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request. Thanks, .NET Foundation Pull Request Bot

[dougbu]

Please let me know if I should split the actual PR comments commit (the middle one) into a separate PR. Would be good to get this repo building on our CI soon.

[dougbu]

Reopening to test out AppVeyor

[dnfclas]

@dougbu, Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request. Thanks, .NET Foundation Pull Request Bot

[rynowak]

init:

Per your request, pausing my reivew until you've have a chance to update :) #Resolved

---

Refers to: .appveyor.yml:1 in 3a50b95. [](commit_id = 3a50b95d546aafc36066c290fed9bed66670b571, deletion_comment = False)

[dougbu]

:up::date:

[dougbu]

Minor :up::date: is just FYI.

[dougbu]

@rynowak please dummy-check the few changes in 1f36dd6e1e. StripeVerifyNotificationIdFilter is about the only class with a significant revision.

[dougbu]

8f0a19bfb9