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

Use the Keycloak server URL for common resources #30823

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

jonkoops
Copy link
Contributor

Changes the bootstrapping for the account console, so that the Keycloak server URL is used (--hostname) to serve common resources, rather than the admin console URL (--hostname-admin).

Closes #30541

@jonkoops jonkoops self-assigned this Jun 26, 2024
@jonkoops jonkoops requested a review from a team as a code owner June 26, 2024 14:16
Copy link
Contributor

@vmuzikar vmuzikar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you for the fix @jonkoops!

@jonkoops jonkoops enabled auto-merge (squash) June 26, 2024 14:27
@jonkoops jonkoops merged commit cd0dbdf into keycloak:main Jun 26, 2024
68 checks passed
@jonkoops jonkoops deleted the server-url-common-resources branch June 26, 2024 14:58
jonkoops added a commit to jonkoops/keycloak that referenced this pull request Jun 26, 2024
jonkoops added a commit to jonkoops/keycloak that referenced this pull request Jun 26, 2024
jonkoops added a commit to jonkoops/keycloak that referenced this pull request Jun 26, 2024
vmuzikar pushed a commit that referenced this pull request Jun 26, 2024
Closes #30541

Signed-off-by: Jon Koops <[email protected]>
(cherry picked from commit cd0dbdf)
vmuzikar pushed a commit that referenced this pull request Jun 27, 2024
Closes #30541

Signed-off-by: Jon Koops <[email protected]>
(cherry picked from commit cd0dbdf)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Account UI resources try to load from admin path instead of frontend path
2 participants