Skip to content

Conversation

thiagoelg
Copy link
Contributor

@thiagoelg thiagoelg commented Mar 11, 2025

Fixes #70

Removes the httpRootPath prefix from quarkus-oidc-proxy routes as Quarkus already does this automatically.

@thiagoelg thiagoelg requested a review from a team as a code owner March 11, 2025 14:50
@sberyozkin
Copy link
Contributor

@thiagoelg I've borrowed this idea from the quarkus webauthn extension.

Can you give me a favor please and add one more test here. Copy OidcProxyTest into OidcProxyHttpRootTest, and have another application-http-root.properties where a custom http root is set, and do .addAsResource("application-http-root.properties", "application.properties") during the test setup.

Thanks

@sberyozkin sberyozkin merged commit 01dc851 into quarkiverse:main Mar 14, 2025
1 check passed
@sberyozkin
Copy link
Contributor

Forgot to press Approve, so confirm it was Approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Root paths are duplicated for quarkus-oidc-proxy routes
2 participants