ai_ml_operations/company_site
2
0 Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Ack Stripe checkouts that belong to another site #36

Merged
westfarn merged 1 commits from ignore-unknown-stripe-checkouts into master 2026-10-30 03:19:49 -07:00
Conversation 0 Commits 1 Files Changed 2
+53 -5
westfarn commented 2026-10-30 03:19:23 -07:00
Owner
Copy link
Copy source

Summary

  • Checkouts from the other site on this Stripe account have no local subscription. The webhook already returned 200 for those; it now logs the ignore.
  • A non-numeric local_subscription_id no longer crashes the lookup.

Test plan

  • uv run python company_site/manage.py test financial.tests.InvoiceAccessTests.test_webhook_ignores_checkout_from_other_site
  • A checkout that matches a local subscription still marks it active.
  • An unrelated checkout posted to the company webhook returns 200.
## Summary - Checkouts from the other site on this Stripe account have no local subscription. The webhook already returned 200 for those; it now logs the ignore. - A non-numeric `local_subscription_id` no longer crashes the lookup. ## Test plan - [ ] `uv run python company_site/manage.py test financial.tests.InvoiceAccessTests.test_webhook_ignores_checkout_from_other_site` - [ ] A checkout that matches a local subscription still marks it active. - [ ] An unrelated checkout posted to the company webhook returns 200.
westfarn added 1 commit 2026-10-30 03:19:23 -07:00
Ack Stripe checkouts that belong to another site.
CI / test (pull_request) Successful in 9s
Details
Unit Tests / test (pull_request) Successful in 9s
Details
0b0ad6b0fa 
Unknown checkout sessions stay a 200, and a non-numeric local id no longer crashes the webhook.
westfarn merged commit 37917c51d2 into master 2026-10-30 03:19:49 -07:00
westfarn deleted branch ignore-unknown-stripe-checkouts 2026-10-30 03:19:49 -07:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
twestfall westfarn (Ryan Westfall)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ai_ml_operations/company_site#36