-
Notifications
You must be signed in to change notification settings - Fork 562
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
[update] 新增stripe聚合支付方式,采用全新的paymentIntents API #136
base: dev
Are you sure you want to change the base?
Conversation
[fix] 修改支付方式中的小bug [update] 将stripe-php版本升级至最新
测试正常 |
啥时候merge一下呀 @cedar2025 |
删除调试冗余代码
运行出现 Class "App\Http\Controllers\V1\Guest\Payment" not found 的提示,订单支付回调正常,在stripe查看日志,一个订单会有两个回调,两个回调日志最大的不同是 "pending_webhooks" 。如果值为3就会提示出错信息,如果值为1则回调成功 |
stripe里监听一个payment_intent.succeeded事件就可以了,我刚刚测试了微信和支付宝两个接口同时存在的情况下,使用支付宝支付成功的时候会向两个接口地址同时发送回调都是成功的,至于第一个not found我看看是啥情况 |
|
not found的问题找到了,webhook失败好像也是这个问题导致的,我看看怎么修复 |
作者你好,求助一下 Webhook 地址尝试填写 都无法正常回调,付款网站收不到通知 |
回调地址是https://域名/api/v1/guest/payment/notify/StripeALLInOne/xxxxxxxx |
谢谢回复,所以我 Webhook 地址填写 https://域名/api/v1/guest/payment/notify/StripeALLInOne/xxxxxxxx 应该无误了 最开始测试的时候,按支付弹出如下错误 将 StripeALLInOne.php 里面的 如果用 Test Mode 会报如下错误 |
这套代码只支持支付宝和微信,暂时不支持信用卡和储蓄卡等其他方式,你可以看看数据库里的log信息的报错。然后根据第二个报错查看,似乎你根本就没有在stripe开启支付宝收款?然后我2套环境测试都是没有问题的,上面好像也有一个老哥测试是正常的 |
误会了,我没有使用信用卡,是报错信息提示他 Test Mode 支持范围只有信用卡和其他... |
请问你们上面提到的 500 报错是在目前的代码已经解决了吗?因为我也有这个报错 浏览器报错路径:/api/v1/user/order/checkout 数据库里的报错记录 |
test mode和正式环境是没差的,按理来说都应该要支持。我这边感觉你是环境的问题,试试使用docker的方式安装呢? |
作者你好,我用了你的镜像部署,但我的数据库是从 wyx2685 那里迁移过来的,在此条件下基本上报错情况都一样 Test Mode 的报错如下 修改那个 suffix 之后就会改成报不支持 alipay,如果作者有空能修也是用户福音,毕竟真钱测试有点肉疼~~ |
建议修改下原v2board中StripeCheckout.php的代码,修改public function notify($params)这一段跟作者的一样,修复由webman引起的回调验签问题。 StripeCheckout能支持你账户的所有付款方式。 我这里经过测试,修改原v2board中三个Stripe的支付文件中回调验证方式,均可正常运行 |
你是不是没更新stripe的api版本和包版本呢,说实话我这里是一点问题也没有的,我也不清楚是啥问题说实话 |
怎么对接哇。全网无教程,那个端点URL我找了半天都不知道在哪。请大佬赐教 |
你仔细看下上面别人的截图~ |
好了。现在我也遇到了支付的时候提示500 Internal Server Error的问题了。 |
@cedar2025 consider merging, please? |
@linusxiong Hi linusxiong, would you consider merging upstream actively since @ cedar2025 is taking a bit longer to make the decision to merge? I am switching to your fork due to the need for Stripe. Not switching to wyx2685/v2board due to the lack of docker support... I would love to buy you a coffee. |
You can just simply use |
Already switched. Thanks! Coffee is still valid 😉 |
[fix] 修复由webman引起的回调验签问题
[fix] 修改支付方式中的小bug
[update] 将stripe-php版本升级至最新
[feat] 新增telegram机器人/start指令监听