-
-
Notifications
You must be signed in to change notification settings - Fork 9k
pay模块使用jodd-http方式,已支持证书方式 #189
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
Conversation
@@ -1,5 +1,7 @@ | |||
package com.github.binarywang.wxpay.service.impl; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个文件没有做实际改动吧?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
没有实际变动,手误提交了
@@ -384,6 +376,7 @@ private String post(String url, String xmlParam) { | |||
|
|||
/** | |||
* 由于暂时未找到使用jodd-http实现证书配置的办法,故而暂时使用httpclient |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
注释能否修正一下?既然已经使用jodd-http实现了,上面的注释就显得无用了吧
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
收到
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
无用注释已去掉
} | ||
HttpRequest request =HttpRequest.post(url).withConnectionProvider(new SSLSocketHttpConnectionProvider(sslContext)); | ||
request.bodyText(requestStr); | ||
HttpResponse response =request.send(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
此行代码貌似没有格式化
} finally { | ||
httpPost.releaseConnection(); | ||
} | ||
HttpRequest request =HttpRequest.post(url).withConnectionProvider(new SSLSocketHttpConnectionProvider(sslContext)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
此行代码貌似没有格式化
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
收到
已格式化代码,再看下 |
无用的注释已去掉 |
1、pay模块使用jodd-http方式,已支持证书方式
2、使用证书的方式和httpclient一样,都是通过java.net.ssl.SSLContext获取机器中的证书