-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
update stripe sdk and webhooks to match #427
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -24,7 +24,7 @@ | |
"react-hot-toast": "^2.4.1", | ||
"react-icons": "4.11.0", | ||
"react-router-dom": "^6.26.2", | ||
"stripe": "11.15.0", | ||
"stripe": "^18.1.0", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. according to new stripe update policy, this will avoid breaking changes. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not sure what this means, can you elaborate? |
||
"tailwind-merge": "^2.2.1", | ||
"tailwindcss": "^3.2.7", | ||
"vanilla-cookieconsent": "^3.0.1", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,6 +5,7 @@ export enum SubscriptionStatus { | |
CancelAtPeriodEnd = 'cancel_at_period_end', | ||
Active = 'active', | ||
Deleted = 'deleted', | ||
Pending = 'pending', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. How come we added this? |
||
} | ||
|
||
export enum PaymentPlanId { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,7 +2,6 @@ import type { StripeMode } from './paymentProcessor'; | |
|
||
import Stripe from 'stripe'; | ||
import { stripe } from './stripeClient'; | ||
import { assertUnreachable } from '../../shared/utils'; | ||
|
||
// WASP_WEB_CLIENT_URL will be set up by Wasp when deploying to production: https://wasp.sh/docs/deploying | ||
const DOMAIN = process.env.WASP_WEB_CLIENT_URL || 'http://localhost:3000'; | ||
|
@@ -41,8 +40,6 @@ export async function createStripeCheckoutSession({ | |
mode, | ||
}: CreateStripeCheckoutSessionParams) { | ||
try { | ||
const paymentIntentData = getPaymentIntentData({ mode, priceId }); | ||
|
||
return await stripe.checkout.sessions.create({ | ||
line_items: [ | ||
{ | ||
|
@@ -54,33 +51,14 @@ export async function createStripeCheckoutSession({ | |
success_url: `${DOMAIN}/checkout?success=true`, | ||
cancel_url: `${DOMAIN}/checkout?canceled=true`, | ||
automatic_tax: { enabled: true }, | ||
allow_promotion_codes: true, | ||
customer_update: { | ||
address: 'auto', | ||
}, | ||
customer: customerId, | ||
// Stripe only allows us to pass payment intent metadata for one-time payments, not subscriptions. | ||
// We do this so that we can capture priceId in the payment_intent.succeeded webhook | ||
// and easily confirm the user's payment based on the price id. For subscriptions, we can get the price id | ||
// in the customer.subscription.updated webhook via the line_items field. | ||
payment_intent_data: paymentIntentData, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we're no longer dealing with payment intents to handle one-time payment products. these are now just getting handled directly in checkout.session.completed. |
||
}); | ||
} catch (error) { | ||
console.error(error); | ||
throw error; | ||
} | ||
} | ||
|
||
function getPaymentIntentData({ mode, priceId }: { mode: StripeMode; priceId: string }): | ||
| { | ||
metadata: { priceId: string }; | ||
} | ||
| undefined { | ||
switch (mode) { | ||
case 'subscription': | ||
return undefined; | ||
case 'payment': | ||
return { metadata: { priceId } }; | ||
default: | ||
assertUnreachable(mode); | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,7 +13,7 @@ import { z } from 'zod'; | |
import { | ||
parseWebhookPayload, | ||
type InvoicePaidData, | ||
type PaymentIntentSucceededData, | ||
SubscriptionCreatedData, | ||
type SessionCompletedData, | ||
type SubscriptionDeletedData, | ||
type SubscriptionUpdatedData, | ||
|
@@ -32,8 +32,8 @@ export const stripeWebhook: PaymentsWebhook = async (request, response, context) | |
case 'invoice.paid': | ||
await handleInvoicePaid(data, prismaUserDelegate); | ||
break; | ||
case 'payment_intent.succeeded': | ||
await handlePaymentIntentSucceeded(data, prismaUserDelegate); | ||
case 'customer.subscription.created': | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. subscription created events used to be sent in customer.subscription.updated events, but they stopped that , so to handle new subscriptions we need this event. |
||
await handleCustomerSubscriptionCreated(data, prismaUserDelegate); | ||
break; | ||
case 'customer.subscription.updated': | ||
await handleCustomerSubscriptionUpdated(data, prismaUserDelegate); | ||
|
@@ -85,67 +85,60 @@ export const stripeMiddlewareConfigFn: MiddlewareConfigFn = (middlewareConfig) = | |
return middlewareConfig; | ||
}; | ||
|
||
// Because a checkout session completed could potentially result in a failed payment, | ||
// we can update the user's payment details here, but confirm credits or a subscription | ||
// if the payment succeeds in other, more specific, webhooks. | ||
// Here we only update the user's payment details, and confirm credits | ||
// if payment mode === payment (e.g. one-time payment). If payment mode === subscription, | ||
// we update its status in the customer.subscription.created or customer.subscription.updated webhook. | ||
// NOTE: If you're accepting async payment methods like bank transfers or SEPA and not just card payments | ||
// which are synchronous, checkout session completed could potentially result in a pending payment. | ||
// If so, use the checkout.session.async_payment_succeeded event to confirm the payment. | ||
export async function handleCheckoutSessionCompleted( | ||
session: SessionCompletedData, | ||
prismaUserDelegate: PrismaClient['user'] | ||
) { | ||
if (session.mode !== 'payment' || session.payment_status !== 'paid') { | ||
return; | ||
} | ||
const userStripeId = session.customer; | ||
const lineItems = await getSubscriptionLineItemsBySessionId(session.id); | ||
|
||
const lineItems = await getCheckoutLineItemsBySessionId(session.id); | ||
const lineItemPriceId = extractPriceId(lineItems); | ||
|
||
const planId = getPlanIdByPriceId(lineItemPriceId); | ||
const plan = paymentPlans[planId]; | ||
if (plan.effect.kind === 'credits') { | ||
return; | ||
} | ||
const { subscriptionPlan } = getPlanEffectPaymentDetails({ planId, planEffect: plan.effect }); | ||
|
||
return updateUserStripePaymentDetails({ userStripeId, subscriptionPlan }, prismaUserDelegate); | ||
const { numOfCreditsPurchased } = getPlanEffectPaymentDetails({ planId, planEffect: plan.effect }); | ||
return updateUserStripePaymentDetails( | ||
{ userStripeId, numOfCreditsPurchased, datePaid: new Date() }, | ||
prismaUserDelegate | ||
); | ||
} | ||
|
||
// This is called when a subscription is purchased or renewed and payment succeeds. | ||
// Invoices are not created for one-time payments, so we handle them in the payment_intent.succeeded webhook. | ||
// This is called when a subscription is successfully purchased or renewed and payment succeeds. | ||
// Invoices are not created for one-time payments, so we handle them above. | ||
export async function handleInvoicePaid(invoice: InvoicePaidData, prismaUserDelegate: PrismaClient['user']) { | ||
const userStripeId = invoice.customer; | ||
const datePaid = new Date(invoice.period_start * 1000); | ||
return updateUserStripePaymentDetails({ userStripeId, datePaid }, prismaUserDelegate); | ||
const lineItems = await invoiceLineItemsSchema.parseAsync(invoice.lines); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not sure if this is needed, because we can be certain that stripe is going to send us the correct format, right? |
||
const priceId = extractPriceId(lineItems); | ||
const subscriptionPlan = getPlanIdByPriceId(priceId); | ||
return updateUserStripePaymentDetails( | ||
{ userStripeId, datePaid, subscriptionPlan, subscriptionStatus: SubscriptionStatus.Active }, | ||
prismaUserDelegate | ||
); | ||
} | ||
|
||
export async function handlePaymentIntentSucceeded( | ||
paymentIntent: PaymentIntentSucceededData, | ||
export async function handleCustomerSubscriptionCreated( | ||
subscription: SubscriptionCreatedData, | ||
prismaUserDelegate: PrismaClient['user'] | ||
) { | ||
// We handle invoices in the invoice.paid webhook. Invoices exist for subscription payments, | ||
// but not for one-time payment/credits products which use the Stripe `payment` mode on checkout sessions. | ||
if (paymentIntent.invoice) { | ||
return; | ||
} | ||
|
||
const userStripeId = paymentIntent.customer; | ||
const datePaid = new Date(paymentIntent.created * 1000); | ||
|
||
// We capture the price id from the payment intent metadata | ||
// that we passed in when creating the checkout session in checkoutUtils.ts. | ||
const { metadata } = paymentIntent; | ||
|
||
if (!metadata.priceId) { | ||
throw new HttpError(400, 'No price id found in payment intent'); | ||
} | ||
|
||
const planId = getPlanIdByPriceId(metadata.priceId); | ||
const plan = paymentPlans[planId]; | ||
if (plan.effect.kind === 'subscription') { | ||
return; | ||
} | ||
|
||
const { numOfCreditsPurchased } = getPlanEffectPaymentDetails({ planId, planEffect: plan.effect }); | ||
|
||
const userStripeId = subscription.customer; | ||
const priceId = extractPriceId(subscription.items); | ||
const subscriptionPlan = getPlanIdByPriceId(priceId); | ||
// We currently use Pending for all other Stripe subscription statuses besides Active that we are not handling. | ||
// If you want to handle these other statuses, make sure to update the `SubscriptionStatus` type in `payment/plans.ts` | ||
const subscriptionStatus: SubscriptionStatus = | ||
subscription.status === SubscriptionStatus.Active | ||
? SubscriptionStatus.Active | ||
: SubscriptionStatus.Pending; | ||
return updateUserStripePaymentDetails( | ||
{ userStripeId, numOfCreditsPurchased, datePaid }, | ||
{ userStripeId, subscriptionPlan, subscriptionStatus }, | ||
prismaUserDelegate | ||
); | ||
} | ||
|
@@ -156,12 +149,11 @@ export async function handleCustomerSubscriptionUpdated( | |
) { | ||
const userStripeId = subscription.customer; | ||
let subscriptionStatus: SubscriptionStatus | undefined; | ||
|
||
const priceId = extractPriceId(subscription.items); | ||
const subscriptionPlan = getPlanIdByPriceId(priceId); | ||
|
||
// There are other subscription statuses, such as `trialing` that we are not handling and simply ignore | ||
// If you'd like to handle more statuses, you can add more cases above. Make sure to update the `SubscriptionStatus` type in `payment/plans.ts` as well | ||
// If you'd like to handle more statuses, you can add more cases above. Make sure to update the `SubscriptionStatus` type in `payment/plans.ts` as well. | ||
if (subscription.status === SubscriptionStatus.Active) { | ||
subscriptionStatus = subscription.cancel_at_period_end | ||
? SubscriptionStatus.CancelAtPeriodEnd | ||
|
@@ -211,17 +203,37 @@ const subscriptionItemsSchema = z.object({ | |
), | ||
}); | ||
|
||
function extractPriceId(items: SubscsriptionItems): string { | ||
type InvoiceLineItems = z.infer<typeof invoiceLineItemsSchema>; | ||
|
||
const invoiceLineItemsSchema = z.object({ | ||
data: z.array( | ||
z.object({ | ||
pricing: z.object({ price_details: z.object({ price: z.string() }) }), | ||
}) | ||
), | ||
}); | ||
|
||
// We only expect one line item, but if you set up a product with multiple prices, you should change this function to handle them. | ||
function extractPriceId(items: SubscsriptionItems | InvoiceLineItems): string { | ||
if (items.data.length === 0) { | ||
throw new HttpError(400, 'No items in stripe event object'); | ||
} | ||
if (items.data.length > 1) { | ||
throw new HttpError(400, 'More than one item in stripe event object'); | ||
} | ||
return items.data[0].price.id; | ||
|
||
const firstItem = items.data[0]; | ||
|
||
if ('price' in firstItem) { | ||
return firstItem.price.id; | ||
} else if ('pricing' in firstItem) { | ||
return firstItem.pricing.price_details.price; | ||
} else { | ||
throw new HttpError(500, 'Unable to extract price id due to unexpected item structure'); | ||
} | ||
} | ||
|
||
async function getSubscriptionLineItemsBySessionId(sessionId: string) { | ||
async function getCheckoutLineItemsBySessionId(sessionId: string) { | ||
try { | ||
const { line_items: lineItemsRaw } = await stripe.checkout.sessions.retrieve(sessionId, { | ||
expand: ['line_items'], | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we link to the file on Github (no permalink, just link to main).
That way, if the file name becomes outdated, the mistake will be easy to spot.