Fix login error : More than 1 route matched the incoming request.

close #60
tchiotludo · May 14, 2019 · 266d120 · 266d120
1 parent f47cec0
commit 266d120
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/main/java/org/kafkahq/controllers/LoginController.java b/src/main/java/org/kafkahq/controllers/LoginController.java
@@ -16,7 +16,7 @@
 @Requires(property = SecurityConfigurationProperties.PREFIX + ".enabled", value = StringUtils.TRUE)
 @Controller
 public class LoginController extends AbstractController {
-    @Get("${kafkahq.server.base-path:}/login{/failed:[a-zA-Z]+}")
+    @Get("${kafkahq.server.base-path:}/login/{failed:[a-zA-Z]+}")
     @View("login")
     public HttpResponse login(Optional<String> failed) {
         return HttpResponse

diff --git a/src/main/java/org/kafkahq/controllers/RedirectController.java b/src/main/java/org/kafkahq/controllers/RedirectController.java
@@ -30,7 +30,7 @@ public HttpResponse home() throws URISyntaxException {
         return HttpResponse.redirect(this.uri("/" + kafkaModule.getClustersList().get(0) + "/topic"));
     }
 
-    @Get("${kafkahq.server.base-path:}/{cluster}")
+    @Get("${kafkahq.server.base-path:}/{cluster:(?!login)[^/]+}")
     public HttpResponse topic(String cluster) throws URISyntaxException {
         return HttpResponse.redirect(this.uri("/" + cluster + "/topic"));
     }