- Issue created by @rszrama
- πΊπΈUnited States rszrama
Note: when this setting is enabled, we should also use it to determine whether or not to log when an incoming webhook is not able to be matched against an existing payment.
- πΊπ¦Ukraine marchuk.vitaliy Rivne, UA
vmarchuk β made their first commit to this issueβs fork.
- last update
over 1 year ago 38 pass - @vmarchuk opened merge request.
- Status changed to Needs review
over 1 year ago 8:05am 10 October 2023 - Status changed to Needs work
over 1 year ago 4:52pm 10 October 2023 - πΊπΈUnited States rszrama
Let's rename the variable to
log_webhooks
to keep the same "verb_object" pattern as the other setting. I also don't see any harm in logging the receipt of any incoming webhook, even if it's of a type the module itself doesn't currently process. Could be useful for debugging or at least optimizing a webhook configuration in a Stripe account. - last update
over 1 year ago 38 pass - last update
over 1 year ago 38 pass - Status changed to Needs review
over 1 year ago 6:42am 11 October 2023 - πΊπ¦Ukraine marchuk.vitaliy Rivne, UA
@rszrama
Let's rename the variable to log_webhooks to keep the same "verb_object" pattern as the other setting. I also don't see any harm in logging the receipt of any incoming webhook, even if it's of a type the module itself doesn't currently process. Could be useful for debugging or at least optimizing a webhook configuration in a Stripe account.
Both points are done.
- Status changed to Closed: outdated
6 months ago 9:04pm 25 August 2024 - πΊπΈUnited States TomTech
Closing this out, since we now support logging webhooks with the webhook events submodule.