Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Stripe Stripe
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 75
    • Issues 75
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 4
    • Merge requests 4
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Extensions
  • StripeStripe
  • Merge requests
  • !17

Webhook System Check

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed bgm requested to merge bgm/stripe:webhook-check into master Jan 03, 2019
  • Overview 2
  • Commits 2
  • Pipelines 0
  • Changes 110

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.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: webhook-check