Skip to content

Run the Vite dev server through the Keycloak server#27311

Merged
jonkoops merged 3 commits into
keycloak:mainfrom
jonkoops:vite-through-keycloak
Jun 12, 2024
Merged

Run the Vite dev server through the Keycloak server#27311
jonkoops merged 3 commits into
keycloak:mainfrom
jonkoops:vite-through-keycloak

Conversation

@jonkoops

@jonkoops jonkoops commented Feb 27, 2024

Copy link
Copy Markdown
Contributor

Changes the development experience for the Admin and Account consoles, so that the applications are served by the Keycloak server directly, which then proxies to the Vite server during development only. This unifies how the the applications are served in production and development, mitigating the need to keep special conditions around during testing.

This PR follows the guidelines from the Vite documentation on custom backend integration, with a couple of slight tweaks specific to our setup.

Closes #19750
Closes #28643
Closes #30115

@jonkoops jonkoops added the status/hold PR should not be merged. On hold for later. label Feb 27, 2024
@jonkoops jonkoops self-assigned this Feb 27, 2024
edewit
edewit previously approved these changes Mar 1, 2024
agagancarczyk
agagancarczyk previously approved these changes Apr 4, 2024
@jonkoops jonkoops force-pushed the vite-through-keycloak branch from c6559ff to 902f456 Compare April 30, 2024 10:32
@jonkoops jonkoops dismissed stale reviews from agagancarczyk and edewit via d37117b April 30, 2024 11:44
@jonkoops jonkoops force-pushed the vite-through-keycloak branch from 902f456 to d37117b Compare April 30, 2024 11:44
This was referenced Apr 30, 2024
@jonkoops jonkoops force-pushed the vite-through-keycloak branch from d37117b to 5500c71 Compare April 30, 2024 11:53
@jonkoops jonkoops force-pushed the vite-through-keycloak branch from 5500c71 to dc795a8 Compare April 30, 2024 14:25
@jonkoops jonkoops force-pushed the vite-through-keycloak branch from dc795a8 to 3cb911d Compare May 21, 2024 12:02
@jonkoops jonkoops dismissed stale reviews from vmuzikar and edewit via facb409 June 11, 2024 14:24
@jonkoops jonkoops requested a review from a team as a code owner June 11, 2024 14:24
@jonkoops jonkoops requested review from edewit and vmuzikar June 11, 2024 14:25
pedroigor
pedroigor previously approved these changes Jun 11, 2024
@pedroigor pedroigor enabled auto-merge (rebase) June 11, 2024 14:25
hmlnarik
hmlnarik previously approved these changes Jun 11, 2024

@hmlnarik hmlnarik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks!

@hmlnarik hmlnarik dismissed their stale review June 11, 2024 15:37

I seem to be reviewing an old version. Will re-review once the tests pass again.

vmuzikar
vmuzikar previously approved these changes Jun 11, 2024
edewit
edewit previously approved these changes Jun 12, 2024
@vmuzikar

Copy link
Copy Markdown
Contributor

@jonkoops Build seems to be failing, though. Can you please check it?

@jonkoops jonkoops dismissed stale reviews from edewit, vmuzikar, and pedroigor via a65d6b3 June 12, 2024 07:48
Signed-off-by: Jon Koops <jonkoops@gmail.com>
@jonkoops jonkoops force-pushed the vite-through-keycloak branch from a65d6b3 to 76df0eb Compare June 12, 2024 07:55
@jonkoops jonkoops disabled auto-merge June 12, 2024 07:55
@jonkoops jonkoops enabled auto-merge (squash) June 12, 2024 07:55

@vmuzikar vmuzikar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Still looks good

@jonkoops jonkoops merged commit c7361cc into keycloak:main Jun 12, 2024

@hmlnarik hmlnarik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great job!

@stianst stianst mentioned this pull request Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

6 participants