Skip to content

Commit b8d011e

Browse files
committed
issue #105 WxCpService.oauth2buildAuthorizationUrl方法设计错误
1 parent 2a439a6 commit b8d011e

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/WxCpService.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -353,10 +353,11 @@ WxMediaUploadResult mediaUpload(String mediaType, String fileType, InputStream i
353353
* 构造oauth2授权的url连接
354354
* 详情请见: http://qydev.weixin.qq.com/wiki/index.php?title=企业获取code
355355
* </pre>
356+
* @param redirectUri
356357
* @param state
357358
* @return code
358359
*/
359-
String oauth2buildAuthorizationUrl(String state);
360+
String oauth2buildAuthorizationUrl(String redirectUri, String state);
360361

361362
/**
362363
* <pre>

weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/WxCpServiceImpl.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -412,10 +412,10 @@ public void tagRemoveUsers(String tagId, List<String> userIds) throws WxErrorExc
412412
}
413413

414414
@Override
415-
public String oauth2buildAuthorizationUrl(String state) {
415+
public String oauth2buildAuthorizationUrl(String redirectUri, String state) {
416416
String url = "https://open.weixin.qq.com/connect/oauth2/authorize?" ;
417417
url += "appid=" + wxCpConfigStorage.getCorpId();
418-
url += "&redirect_uri=" + URIUtil.encodeURIComponent(wxCpConfigStorage.getOauth2redirectUri());
418+
url += "&redirect_uri=" + URIUtil.encodeURIComponent(redirectUri);
419419
url += "&response_type=code";
420420
url += "&scope=snsapi_base";
421421
if (state != null) {

weixin-java-cp/src/test/java/me/chanjar/weixin/cp/demo/WxCpDemoServer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ public WxCpXmlOutMessage handle(WxCpXmlMessage wxMessage, Map<String, Object> co
6262
@Override
6363
public WxCpXmlOutMessage handle(WxCpXmlMessage wxMessage, Map<String, Object> context,
6464
WxCpService wxCpService, WxSessionManager sessionManager) {
65-
String href = "<a href=\"" + wxCpService.oauth2buildAuthorizationUrl(null)
65+
String href = "<a href=\"" + wxCpService.oauth2buildAuthorizationUrl(wxCpConfigStorage.getOauth2redirectUri(), null)
6666
+ "\">测试oauth2</a>";
6767
return WxCpXmlOutMessage
6868
.TEXT()

0 commit comments

Comments
 (0)