CB-5559 include root uri in redirect urls (#2950)

Co-authored-by: Daria Marutkina <[email protected]>
dbeaver · Oct 1, 2024 · 3fbc227 · 3fbc227
1 parent bb9a62d
commit 3fbc227
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/...undles/io.cloudbeaver.model/src/io/cloudbeaver/registry/WebAuthProviderConfiguration.java b/...undles/io.cloudbeaver.model/src/io/cloudbeaver/registry/WebAuthProviderConfiguration.java
@@ -80,7 +80,7 @@ public String getSignInLink() throws DBException {
     }
 
     private String buildRedirectUrl(String baseUrl) {
-        return baseUrl + "?" + CBAuthConstants.CB_REDIRECT_URL_REQUEST_PARAM + "=" + WebAppUtils.getWebApplication().getServerURL();
+        return baseUrl + "?" + CBAuthConstants.CB_REDIRECT_URL_REQUEST_PARAM + "=" + WebAppUtils.getFullServerUrl();
     }
 
     @Property

diff --git a/server/bundles/io.cloudbeaver.model/src/io/cloudbeaver/utils/WebAppUtils.java b/server/bundles/io.cloudbeaver.model/src/io/cloudbeaver/utils/WebAppUtils.java
@@ -38,6 +38,8 @@
 
 import java.nio.file.Path;
 import java.util.*;
+import java.util.stream.Collectors;
+import java.util.stream.Stream;
 
 public class WebAppUtils {
     private static final Log log = Log.getLog(WebAppUtils.class);
@@ -269,4 +271,13 @@ private static void flattenArray(Object[] array, Map<String, Object> result, Str
         }
     }
 
+    @NotNull
+    public static String getFullServerUrl() {
+        WebApplication application = WebAppUtils.getWebApplication();
+        return Stream.of(application.getServerURL(), application.getRootURI())
+            .map(WebAppUtils::removeSideSlashes)
+            .filter(CommonUtils::isNotEmpty)
+            .collect(Collectors.joining("/"));
+    }
+
 }