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

QQHandler类中,openId参数值没有正确使用的问题 #185

Open
andywu188 opened this issue Sep 4, 2020 · 0 comments
Open

QQHandler类中,openId参数值没有正确使用的问题 #185

andywu188 opened this issue Sep 4, 2020 · 0 comments
Labels
Bug 🐛 有虫,来捉虫
Milestone

Comments

@andywu188
Copy link

描述一下BUG

OSharp.Identity.OAuth2.QQ.QQHandler类中,第60行:

`

        string json2 = await userInformationResponse.Content.ReadAsStringAsync();
        JObject payload = JObject.Parse(json2);
        JsonDocument document = JsonDocument.Parse(json2);
        //identity.AddClaim(new Claim(ClaimTypes.NameIdentifier, openId, ClaimValueTypes.String, Options.ClaimsIssuer));

        payload.Add("openid", openId);//注意这里重点

        OAuthCreatingTicketContext context = new OAuthCreatingTicketContext(new ClaimsPrincipal(identity), properties, Context, Scheme, Options, Backchannel, tokens, document.RootElement);

`
以上代码中,payload.Add("openid", openId),之后,没有任何地方有使用payload,应该是有BUG,正确代码应该是向JsonDocument中增加键值对吧

运行环境

  • Version 3.1.7
@gmf520 gmf520 added the Bug 🐛 有虫,来捉虫 label Sep 7, 2020
@gmf520 gmf520 added this to the vNext milestone Sep 7, 2020
@gmf520 gmf520 modified the milestones: v5.0.3, vNext Feb 27, 2021
@gmf520 gmf520 modified the milestones: v5.0.4, vNext Apr 2, 2021
@gmf520 gmf520 modified the milestones: v6.0.0, vNext Dec 27, 2021
@gmf520 gmf520 modified the milestones: v6.0.5, vNext Apr 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐛 有虫,来捉虫
Projects
None yet
Development

No branches or pull requests

2 participants