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

修复两个bug #301

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from
Open

修复两个bug #301

wants to merge 12 commits into from

Conversation

towserchen
Copy link

1, 官方签发的应用证书在 -----END CERTIFICATE----- 前可能存在换行,导致 php openssl_x509_parse 函数无法识别证书内容,提交请求时支付宝回复”应用公钥证书不存在“错误。新增清理无用空行函数,并在读取证书时应用,以解决该问题;
2, AntCertificationUtil->hex2dec() 方法可能因为用户应用在其他位置修改了 bcscale 精度而导致返回值存在多余的小数位0。我所遭遇的情况更神奇:因为测试环境配置问题 php-fpm 没能及时重启进程,当请求分配到不同的 php 进程上时 bcscale 返回都不同。新增在 hex2dec 函数内强制配置 bcscale 为0 ,并在执行完后恢复用户之前的 scale 值,顺利解决这个问题;

@ydslash2
Copy link

借个地方吐槽下阿里,支付页面连个orderId都要藏起来,真向58学习了呢

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

Successfully merging this pull request may close these issues.

5 participants