Prevent frontend endpoint redirect to admin endpoint #38464
Conversation
| * @author Vaclav Muzikar <vmuzikar@redhat.com> | ||
| */ | ||
| @KeycloakIntegrationTest(config = AdminRootTest.AdminUrlConfig.class) | ||
| public class AdminRootTest { |
There was a problem hiding this comment.
Not sure if similar test exists in the old testsuite but seemed simpler to do it in the new one as we need to provide server config options. Let me know if that works for you.
| @Override | ||
| public LifeCycle getDefaultLifecycle() { | ||
| return LifeCycle.GLOBAL; | ||
| } |
There was a problem hiding this comment.
This might be a bit controversial but I assume we might need occasionally to provide some HTTP Client configs on the test level, hence the global scope would not work. Let me know if that works for you.
There was a problem hiding this comment.
Don't change this, instead add the compatible check based on annotations. That'll recreate the global instance as needed based on annotation config.
There was a problem hiding this comment.
Thanks for the pointer. I didn't realize the compatible method is there for this use case. Created #38586.
861a258 to
e02fb9a
Compare
ea5297a to
796
8000
8b8f
Compare
Closes keycloak#38463 Signed-off-by: Václav Muzikář <vmuzikar@redhat.com>
7968b8f to
2a52dff
Compare
Closes #38463