Skip to content

Webhook System Check

bgm requested to merge bgm/stripe:webhook-check into master

This PR proposes two things:

  • A CiviCRM System Check to make sure that the live Stripe processors (it ignores test ones) have a valid webhook configured.
  • To implement this, I had to bump the stripe-php library to the latest version (WebhookEndpoint was added to stripe-php in October 2018).

I implemented this because it's rather easy to do an error while configuring the webhook (forgetting the 'www', 'https' or using the wrong ID). It also catches incorrect configurations for those upgrading from earlier versions that used the old webhook URL.

Ideally the next step would be to offer to create the webhook automatically.

Merge request reports