Skip to content

fix(stripe): enforce paymentMethodTypes allowlist#21

Open
stevencartavia wants to merge 1 commit into
stripe:mainfrom
stevencartavia:fix/stripe-payment-method-types-allowlist
Open

fix(stripe): enforce paymentMethodTypes allowlist#21
stevencartavia wants to merge 1 commit into
stripe:mainfrom
stevencartavia:fix/stripe-payment-method-types-allowlist

Conversation

@stevencartavia
Copy link
Copy Markdown

The Stripe method advertised an allowed-payment-methods list but never enforced it.

  • Challenge now uses the spec key paymentMethodTypes (was paymentMethods)
  • verify sends payment_method_types to Stripe instead of automatic_payment_methods, so only allowed methods can charge
  • payment_methods: is now required (raises ArgumentError if missing/empty)

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Jun 5, 2026

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Jun 5, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@stevencartavia stevencartavia force-pushed the fix/stripe-payment-method-types-allowlist branch from cb4b6f0 to 8d7f4b1 Compare June 6, 2026 03:27
Signed-off-by: steven <corderosteven6@gmail.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019e99f9-eb58-739a-bb39-019910d859c4
Co-authored-by: Amp <amp@ampcode.com>
@stevencartavia stevencartavia force-pushed the fix/stripe-payment-method-types-allowlist branch from 8d7f4b1 to c0f86d1 Compare June 6, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant