From 6739e98662c16571da7aec0e9b52fab0afd3833b Mon Sep 17 00:00:00 2001
From: sunpengfei <i@angelzzz.com>
Date: 星期五, 13 六月 2025 14:07:28 +0800
Subject: [PATCH] fix:修订支付回调和供应商回调并发问题

---
 LifePayment/LifePayment.HttpApi/LifePay/WxPayNotifyController.cs |   15 ++++++++++++---
 1 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/LifePayment/LifePayment.HttpApi/LifePay/WxPayNotifyController.cs b/LifePayment/LifePayment.HttpApi/LifePay/WxPayNotifyController.cs
index de9e04b..9e30708 100644
--- a/LifePayment/LifePayment.HttpApi/LifePay/WxPayNotifyController.cs
+++ b/LifePayment/LifePayment.HttpApi/LifePay/WxPayNotifyController.cs
@@ -1,6 +1,7 @@
 锘縰sing LifePayment.Application.Contracts;
 using LifePayment.Domain;
 using LifePayment.Domain.Shared;
+using Medallion.Threading;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Caching.Distributed;
 using Microsoft.Extensions.Logging;
@@ -10,6 +11,7 @@
 using System.Threading.Tasks;
 using Volo.Abp.AspNetCore.Mvc;
 using Volo.Abp.Caching;
+using Volo.Abp.DistributedLocking;
 using Volo.Abp.Uow;
 using ZeroD.Util;
 
@@ -26,6 +28,7 @@
         private readonly ILifePayService _lifePayService;
         private readonly ILifePayOrderService _lifePayOrderService;
         private readonly IDistributedCache<string> distributedCache;
+        private readonly IAbpDistributedLock distributedLock;
         private readonly ILogger<WxPayNotifyController> _logger;
 
         public WxPayNotifyController(
@@ -33,13 +36,15 @@
                ILogger<WxPayNotifyController> logger,
                ILifePayService lifePayService,
                ILifePayOrderService lifePayOrderService,
-               IDistributedCache<string> distributedCache)
+               IDistributedCache<string> distributedCache,
+               IAbpDistributedLock distributedLock)
         {
             _wxPayApi = wxPayApi;
             _logger = logger;
             _lifePayService = lifePayService;
             _lifePayOrderService = lifePayOrderService;
             this.distributedCache = distributedCache;
+            this.distributedLock = distributedLock;
         }
 
         /// <summary>
@@ -48,7 +53,6 @@
         /// <param name="input"></param>
         /// <returns></returns>
         [HttpPost]
-        [UnitOfWork]
         public async Task<WxRechargeNotifyResult> WxRechargeNotify(WxRechargeNotifyInput input)
         {
             try
@@ -59,10 +63,14 @@
                 _logger.LogInformation($"鐢熸椿绠″寰俊鍏呭�煎洖璋冮�氱煡data锛�" + data);
                 var wxPayNotice = JsonConvert.DeserializeObject<WxPayNotice>(data);
 
+                await using var orderLock = await distributedLock.TryAcquireAsync($"LockKey:UpdateOrder:{wxPayNotice.OutTradeNo}", TimeSpan.FromSeconds(60));
+                _logger.LogInformation($"閿侊細LockKey:UpdateOrder:{wxPayNotice.OutTradeNo} - {orderLock != null}");
+
                 if (wxPayNotice.OutTradeNo.Contains("JF"))
                 {
                     if (wxPayNotice.TradeState == LifePaymentConstant.WxPayStatus.鏀粯鎴愬姛)
                     {
+
                         var key = $"WxRechargeNotify_{wxPayNotice.OutTradeNo}";
                         if (string.IsNullOrWhiteSpace(distributedCache.Get(key)))
                         {
@@ -85,6 +93,8 @@
                             {
                                 AbsoluteExpirationRelativeToNow = TimeSpan.FromMinutes(10)
                             });
+
+                            await CurrentUnitOfWork.SaveChangesAsync();
                         }
                         else
                         {
@@ -103,7 +113,6 @@
             }
             catch (Exception ex)
             {
-                await CurrentUnitOfWork.RollbackAsync();
                 return new WxRechargeNotifyResult
                 {
                     Code = "FAIL",

--
Gitblit v1.9.1