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.

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

Jump to bottom

Wrong token endpoint value caused "oauth2: server response missing access_token" #314

Closed
leopku opened this issue Nov 8, 2021 · 1 comment · Fixed by #315
Closed

Wrong token endpoint value caused "oauth2: server response missing access_token" #314

leopku opened this issue Nov 8, 2021 · 1 comment · Fixed by #315
Assignees
@hsluoyz
Labels
bug Something isn't working

Comments

@leopku
Copy link
Contributor

leopku commented Nov 8, 2021

I got "oauth2: server response missing access_token" while connecting casdoor using third-party oauth client.
But it's ok while changing casdoor to Keycloak.

After digging into source code, I found a wrong value of TokenEndpoint.
Changing TokenEndpoint: fmt.Sprintf("%s/api/login", oidcOrigin) to TokenEndpoint: fmt.Sprintf("%s/api/login/oauth/access_token", oidcOrigin), would make all thing works well.
leopku pushed a commit to leopku/casdoor that referenced this issue Nov 8, 2021
@antbox
fix: Wrong token endpoint value casdoor#314
e53cc0c
@leopku leopku mentioned this issue Nov 8, 2021
Merged
leopku pushed a commit to leopku/casdoor that referenced this issue Nov 8, 2021
@antbox
fix: Wrong token endpoint value casdoor#314
0b8ff8f 
Signed-off-by: leo <leo@himysql.com>
hsluoyz pushed a commit that referenced this issue Nov 8, 2021
@leopku @antbox
fix: Wrong token endpoint value #314 (#315)
d84ddda 
Signed-off-by: leo <leo@himysql.com>

Co-authored-by: leo <leo@himysql.com>
@casbin-bot
Copy link
Contributor

@seriouszyx @tangyang9464

@casbin-bot casbin-bot added the bug Something isn't working label Nov 8, 2021
@casdoor casdoor deleted a comment from casbin-bot Nov 8, 2021
hyperzlib pushed a commit to hyperzlib/casdoor that referenced this issue Nov 10, 2021
@leopku @antbox @hyperzlib
fix: Wrong token endpoint value casdoor#314 (casdoor#315)
a1cf283 
Signed-off-by: leo <leo@himysql.com>

Co-authored-by: leo <leo@himysql.com>
Signed-off-by: Lex Lim <hyperzlink@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hsluoyz hsluoyz

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Wrong token endpoint value #314 leopku/casdoor
3 participants
@leopku @hsluoyz @casbin-bot