Is your feature request related to a problem? Please describe.
If a shop fails to announce itself,
- 55faed5 (Now shops always attempt to announce upon first payment provider (e.g. stripe) account creation and don't rely depend on external uwsgi cron call)
|
def announce_shop_stripe_connect_ids(): |
Then there will be no entry for the shop , and https://github.com/Subscribie/stripe-connect-webhook-endpoint-router will not be able to route webhooks to it:
https://github.com/Subscribie/stripe-connect-webhook-endpoint-router/blob/4216d104334cf65c684f1dc65b6695e1972a7f29/main.py#L56
Describe the solution you'd like
Add health check so that shop can self-report if it's not been able to announce it's address
Describe alternatives you've considered
- Announcing every 5 mintues works well (
|
cron2 = minute=-5 curl -L %(vassal_name)\/admin\/announce-stripe-connect |
) , however, in Playwright (test) environments uwsgi isn't present so CI must announce
- drop the use of
uwsgi cron (uwsgi cron no longer needed at all - for first time announce at least, see 55faed5 )
- incoporate
uwsgi cron into dokku deployments
- use background thread over cron
- use/reply on basic cron similar to Drupals built-in best-effort cron
Additional context
Is your feature request related to a problem? Please describe.
If a shop fails to announce itself,
subscribie/subscribie/blueprints/admin/__init__.py
Line 1928 in 6c1d968
Then there will be no entry for the shop , and https://github.com/Subscribie/stripe-connect-webhook-endpoint-router will not be able to route webhooks to it:
https://github.com/Subscribie/stripe-connect-webhook-endpoint-router/blob/4216d104334cf65c684f1dc65b6695e1972a7f29/main.py#L56
Describe the solution you'd like
Add health check so that shop can self-report if it's not been able to announce it's address
Describe alternatives you've considered
subscribie/vassals-inject-config.ini
Line 23 in 6c1d968
uwsgiisn't present so CI must announceuwsgicron (uwsgi cron no longer needed at all - for first time announce at least, see 55faed5 )uwsgicron into dokku deploymentsAdditional context