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

Space in name of authentication source causes problems for Keycloak #12647

Closed
2 of 7 tasks
jcollie opened this issue Aug 30, 2020 · 2 comments · Fixed by #12648
Closed
2 of 7 tasks

Space in name of authentication source causes problems for Keycloak #12647

jcollie opened this issue Aug 30, 2020 · 2 comments · Fixed by #12648
Labels
Milestone

Comments

@jcollie
Copy link

jcollie commented Aug 30, 2020

  • Gitea version (or commit ref): 1.12.3
  • Git version: 2.24.3
  • Operating system: Docker
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

I created an OAuth2/OpenID Connect authentication source to test out authentication with my new Keycloak server. When assigning an Authentication Name I used "keycloak test". This appears to have caused problems with Keycloak because of the space in the name. Changing the space to a dash character in Gitea fixes the error. I would think that url-encoding the space would have fixed the problem as well. Here's the error that I see in Keycloak's log:

03:07:45,157 WARN [org.keycloak.events] (default task-38) type=LOGIN_ERROR, realmId=test, clientId=gitea, userId=null, ipA
ddress=127.0.0.1, error=invalid_redirect_uri, redirect_uri='https://git.example.com/user/oauth2/keycloak test/callback'

@zeripath
Copy link
Contributor

Thanks for alerting us.

@jcollie
Copy link
Author

jcollie commented Aug 31, 2020

Thanks for fixing so quickly!

@lafriks lafriks added this to the 1.12.4 milestone Aug 31, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants