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

Minor OIDC devui updates #35593

Merged
merged 2 commits into from
Aug 28, 2023
Merged

Conversation

sberyozkin
Copy link
Member

Minor updates to the DevUI after testing it with Auth0:

  • DevUI script should include openid scope if it is not explicitly configured, example, with quarkus.oidc.authentication.scopes=profile. It has always been the case with the Quarkus OIDC itself, but in DevUI it is not currently done. I've found out about it after successfully testing a Quarkus OIDC endpoint with quarkus.oidc.authentication.scopes=profile and then spending a couple of hours trying to figure why in DevUI SPA, IdToken is not returned from Auth0 - turned out the openid scope was missing - it can be easily fixed with quarkus.oidc.authentication.scopes=openid,profile but users should not spend time chasing it if they omit openid
  • For redirect from OIDC providers back to DevUI SPA to work, in many case users would have to register a DevUI page address in the provider's console. The currently documented address has changed so I fixed it in the docs. (it works with the default Keycloak devservice launch because a wildcard redirect is enabled)

@github-actions
Copy link

github-actions bot commented Aug 28, 2023

🙈 The PR is closed and the preview is expired.

@quarkus-bot
Copy link

quarkus-bot bot commented Aug 28, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@sberyozkin sberyozkin merged commit 193e629 into quarkusio:main Aug 28, 2023
@sberyozkin sberyozkin deleted the minor_oidc_devui_updates branch August 28, 2023 13:59
@quarkus-bot quarkus-bot bot added this to the 3.4 - main milestone Aug 28, 2023
@gsmet gsmet modified the milestones: 3.4 - main, 3.3.2 Sep 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants