yml2213andCursor
590bf1cae3
租客在待支付超时或取消的最后几十秒完成付款时，订单先被取消，乐刷的支付 结果几秒后才到，入账被判定订单不可支付而拒收。原先订单推进和支付单更新 在同一个事务里，订单一报错整笔回滚，支付单停在支付中且 paid_at 为空： 钱已在渠道，系统却毫不知情。乐刷重试十次后放弃，之后既补不回入账（订单 不再是待支付），后台人工退款也发不出去（退款要求原支付单是已支付）。 日志保留期内 13 天出了 5 笔，合计 4085.60 元。 - 取消前核对渠道：超时取消、租客取消、客服关单都先查一次渠道，已收款并 已推进订单就放弃取消，让支付正常入账而不是走退款。渠道异常时按未收款 继续，避免渠道一抖订单全卡住 - 回调兜底记账：订单确实已结束时不再回滚，把支付单如实记成已支付并写入 paid_at 与渠道单号，后台人工退款从此可用。退款仍由客服人工发起 - 客服关单：待支付订单按实际已收款减已退款的差额退，补上本次事故中关单 不生成退款的直接原因 - 补齐日志上下文：回调入账失败原先只有 error="order cannot pay"，现在带 支付单号、订单号、订单状态和金额；新增 ERROR「渠道已收款但订单无法入账， 需人工退款」作为唯一需人工介入的告警 Co-authored-by: Cursor <cursoragent@cursor.com>
90 lines
3.3 KiB
Go
90 lines
3.3 KiB
Go
package payment
|
|
|
|
import (
|
|
"context"
|
|
|
|
"hfb_sys/backend/internal/model"
|
|
|
|
"go.uber.org/zap"
|
|
)
|
|
|
|
// ConfirmChannelPaidBeforeClose 在取消或关闭订单前跟渠道核对未完成的下单支付单。
|
|
// 返回 true 表示渠道确认已收款且订单已被推进，调用方必须放弃本次取消。
|
|
//
|
|
// 待支付订单的 15 分钟超时和用户手动取消都可能正好撞上用户付款：
|
|
// 订单先没了，渠道结果几秒后才到，于是钱收了、订单却是已取消。
|
|
// 先核对一次能把绝大多数这种竞态挡在发生之前——而且是让支付正常入账，不必退款。
|
|
func (r *Repository) ConfirmChannelPaidBeforeClose(ctx context.Context, orderID uint64) (bool, error) {
|
|
if r == nil || r.db == nil || orderID == 0 {
|
|
return false, nil
|
|
}
|
|
payments, err := r.pendingOrderPayments(ctx, orderID)
|
|
if err != nil {
|
|
return false, err
|
|
}
|
|
advanced := false
|
|
for i := range payments {
|
|
if err := r.probeChannelPaid(ctx, &payments[i]); err != nil {
|
|
// 渠道不可用不能阻塞取消，否则渠道一挂订单就全卡住。
|
|
// 真漏掉的由支付回调兜底记账并告警。
|
|
r.log().Warn("取消前核对渠道支付状态失败", paymentLogFields(ctx, appendFields(
|
|
paymentOrderFields(&payments[i]),
|
|
[]zap.Field{zap.Error(err)},
|
|
)...,
|
|
)...)
|
|
continue
|
|
}
|
|
// 以订单是否真的被推进为准。钱到了但商品已被他人租走时，入账路径只会记账
|
|
// 并告警等人工退款，不推进订单，这种情况仍然该让取消照常进行。
|
|
if orderAdvancedPastPayment(r.orderStatus(ctx, orderID)) {
|
|
advanced = true
|
|
}
|
|
}
|
|
return advanced, nil
|
|
}
|
|
|
|
// orderAdvancedPastPayment 判断订单是否已经靠这笔支付走出待支付阶段。
|
|
func orderAdvancedPastPayment(status string) bool {
|
|
return status == "pending_handoff" || status == "renting"
|
|
}
|
|
|
|
// probeChannelPaid 向渠道查询一笔支付单的真实状态并落库。
|
|
func (r *Repository) probeChannelPaid(ctx context.Context, payment *model.PaymentOrder) error {
|
|
if payment.ProviderOrderID == "" {
|
|
// 没有渠道单号说明下单请求没被渠道受理，不可能收到钱。
|
|
return nil
|
|
}
|
|
runtimeConfig, err := r.runtimeConfigForPayment(ctx, payment)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if runtimeConfig.isMockMode() || runtimeConfig.Channel == nil {
|
|
return nil
|
|
}
|
|
resp, err := runtimeConfig.Channel.QueryPayment(ctx, payment.ThirdOrderID, payment.ProviderOrderID)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if resp.Status != "paid" {
|
|
return nil
|
|
}
|
|
r.log().Warn("渠道核对到已收款但系统未入账", paymentLogFields(ctx, appendFields(
|
|
paymentOrderFields(payment),
|
|
runtimeConfigFields(runtimeConfig),
|
|
[]zap.Field{zap.String("order_status", r.orderStatus(ctx, payment.OrderID))},
|
|
)...,
|
|
)...)
|
|
// 走统一入账路径：订单还能支付就正常推进，已经结束就只记账并告警等人工退款。
|
|
return r.applyChannelStatus(ctx, payment, resp.Status, resp.PayTime, resp.Raw, channelSourceQuery)
|
|
}
|
|
|
|
func (r *Repository) pendingOrderPayments(ctx context.Context, orderID uint64) ([]model.PaymentOrder, error) {
|
|
var rows []model.PaymentOrder
|
|
err := r.db.WithContext(ctx).
|
|
Where("order_id = ? AND biz_type = ? AND status IN ? AND provider_order_id <> ''",
|
|
orderID, "order_pay", []string{"created", "paying"}).
|
|
Order("id DESC").
|
|
Find(&rows).Error
|
|
return rows, err
|
|
}
|