Skip to content

Increase support for authorization of Wechat Webpage #262

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.

Sign up for GitHub

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

Merged
merged 13 commits into from
May 22, 2019
Merged

Increase support for authorization of Wechat Webpage #262

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
83213dd
Introduce a force_verify option in the Twitch provider
zAfLu Dec 4, 2018
05536cf
Merge branch 'dev' of https://github.com/saber-wang/AspNet.Security.O…
saber-wang Jan 9, 2019
ecf0e60
Merge branch 'dev' of https://github.com/saber-wang/AspNet.Security.O…
saber-wang Jan 9, 2019
d2933d8
Merge branch 'dev' of https://github.com/saber-wang/AspNet.Security.O…
saber-wang Jan 14, 2019
de50069
Store state in redirectUri when authorizing Wechat Web pages to preve…
saber-wang May 15, 2019
ba6e81c
Optimization code
saber-wang May 15, 2019
17fd49d
Use TryGetValue()
saber-wang May 17, 2019
fa0298d
Merge pull request #1 from aspnet-contrib/dev
saber-wang May 20, 2019
7891b25
add WechatTests
saber-wang May 20, 2019
53320b1
Revert "add WechatTests"
saber-wang May 20, 2019
7ba0fc6
Revert "Revert "add WechatTests""
saber-wang May 20, 2019
9a0856c
Retain the _oauthstate parameter in redirect_uri
saber-wang May 20, 2019
41426f9
Refactor _oauthstate handling
martincostello May 22, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Store state in redirectUri when authorizing Wechat Web pages to preve… 
…nt "too long state parameters" error and Add the #wechat redirect parameter
  • Loading branch information
@saber-wang
saber-wang committed May 15, 2019
commit de500696c80bb0bcc0ce1bd61386479b23323d5c
4 changes: 2 additions & 2 deletions src/AspNet.Security.OAuth.Weixin/WeixinAuthenticationHandler.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,9 +129,9 @@ protected override string BuildChallengeUrl(AuthenticationProperties properties,
var oauthstate = Options.StateDataFormat.Protect(properties);
if (Options.AuthorizationEndpoint != WeixinAuthenticationDefaults.AuthorizationEndpoint)
{
//微信网页授权时将state存放到redirectUri中，防止"state参数过长"错误
//Store state in redirectUri when authorizing Wechat Web pages to prevent "too long state parameters" error
redirectUri = redirectUri.Contains("?") ? $"{redirectUri}&{nameof(oauthstate)}={oauthstate}" : $"{redirectUri}?{nameof(oauthstate)}={oauthstate}";
oauthstate = "#wechat_redirect";//微信网页授权支持
oauthstate = "#wechat_redirect";//The Parameters Necessary for Web Authorization of Wechat
}
return QueryHelpers.AddQueryString(Options.AuthorizationEndpoint, new Dictionary<string, string>
{
Expand Down