Skip to content
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

微信支付沙箱环境URL错误 #627

Closed
abcfy2 opened this issue Jun 13, 2018 · 7 comments
Closed

微信支付沙箱环境URL错误 #627

abcfy2 opened this issue Jun 13, 2018 · 7 comments

Comments

@abcfy2
Copy link

abcfy2 commented Jun 13, 2018

参见微信支付沙箱环境文档

微信支付沙箱环境的URL prefix是/sandbox/,而不是/sandboxnew/,旧的代码会返回502,待会我会递交一个PR修复这个问题

@abcfy2
Copy link
Author

abcfy2 commented Jun 13, 2018

好像不能简单的这么改,微信支付沙箱用的是/sandbox,而刷卡仿真用的却是/sandboxnew。

微信的开发文档绝对是我见过的最烂的开发文档

@binarywang
Copy link
Owner

我去,居然没发现这个问题,主要是之前这个环境一直没法用

@abcfy2
Copy link
Author

abcfy2 commented Jun 13, 2018

现在可以用了,我在本地修改代码build之后推送到maven local,这个接口就正常了,修改weixin-java-pay/src/main/java/com/github/binarywang/wxpay/service/impl/BaseWxPayServiceImpl.java第81行,

      return PAY_BASE_URL + "/sandbox";

之后在本地测试就过了。但是我只测试了微信支付,所以不确定这个影响会不会干扰到其他sandbox接口,需要系统测试一下。微信居然沙箱环境用的prefix都不统一,真是醉了。而且微信支付沙箱环境文档的那个配图和底下的文字描述根本不相符。

微信的开发文档确实稀烂,官方还没有SDK,开发接入非常麻烦,和支付宝的开发体验差远了。

@abcfy2
Copy link
Author

abcfy2 commented Jun 13, 2018

按照这个issue: w7corp/easywechat#665

似乎/sandboxnew才是对的,但是需要签名,这里可能得在sdk改一下,否则返回如下内容:

【响应数据】:<xml>
  <return_code><![CDATA[FAIL]]></return_code>
  <return_msg><![CDATA[沙箱验证签名失败,请确认沙箱签名key是否正确(通过getsignkey调用生成)]]></return_msg>
</xml>

@wangtao
Copy link

wangtao commented Jun 13, 2018

我看的文档是 https://pay.weixin.qq.com/wiki/doc/api/jsapi.php?chapter=23_1

图文都是 sandboxnew

@abcfy2
Copy link
Author

abcfy2 commented Jun 13, 2018

微信自己的文档就很凌乱,很多文档都不更新,文档中有一大堆死链也没人修复。

之前沙箱环境的URL是对的,但是按照微信文档,必须先getsignkey,建议在sdk处理下这个。可以参考我上面贴出来的issue链接

@binarywang
Copy link
Owner

那其实也问题不大,本身沙箱环境就是建议先调用getsignkey来生成signkey,然后再来调用的。这个接口我们也提供了,需要使用者自己手动去掉用即可

@abcfy2 abcfy2 closed this as completed Jun 13, 2018
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

No branches or pull requests

3 participants