fix: extract browser details earlier (#23412)

Extract the browserDetails earlier
so that they are available when
UiInitListeners are fired.

Fixes #23396

Author: caalador

flow-server/src/main/java/com/vaadin/flow/server/BootstrapHandler.java

@@ -59,6 +59,7 @@
 
 import com.vaadin.flow.component.PushConfiguration;
 import com.vaadin.flow.component.UI;
+import com.vaadin.flow.component.page.ExtendedClientDetails;
 import com.vaadin.flow.component.page.Inline;
 import com.vaadin.flow.component.page.Push;
 import com.vaadin.flow.component.page.Viewport;
@@ -1354,6 +1355,9 @@ protected BootstrapContext createAndInitUI(Class<? extends UI> uiClass,
 
         push.map(Push::transport).ifPresent(pushConfiguration::setTransport);
 
+        // Parse browser details from request parameters and store in UI
+        extractAndStoreBrowserDetails(request, ui);
+
         // Set thread local here so it is available in init
         UI.setCurrent(ui);
         ui.doInit(request, session.getNextUIid(), context.getAppId());
@@ -1367,6 +1371,34 @@ protected BootstrapContext createAndInitUI(Class<? extends UI> uiClass,
         return context;
     }
 
+    /**
+     * Extracts browser details from the request JSON parameter and stores them
+     * in the UI's internals as ExtendedClientDetails.
+     *
+     * @param request
+     *            the request containing browser details as JSON parameter
+     * @param ui
+     *            the UI instance to store the details in
+     */
+    private void extractAndStoreBrowserDetails(VaadinRequest request, UI ui) {
+        // Extract browser details JSON parameter from request
+        // This is sent by the client in the v-r=init request
+        String browserDetailsJson = request.getParameter("v-browserDetails");
+
+        if (browserDetailsJson != null && !browserDetailsJson.isEmpty()) {
+            try {
+                JsonNode json = JacksonUtils.readTree(browserDetailsJson);
+                ExtendedClientDetails details = ExtendedClientDetails
+                        .fromJson(ui, json);
+                ui.getInternals().setExtendedClientDetails(details);
+            } catch (Exception e) {
+                // Log and continue without browser details
+                getLogger().debug(
+                        "Failed to parse browser details from init request", e);
+            }
+        }
+    }
+
     protected void initializeUIWithRouter(BootstrapContext context, UI ui) {
         if (ui.getInternals().getRouter() != null) {
             ui.getInternals().getRouter().initializeUI(ui, context.getRoute());

flow-server/src/main/java/com/vaadin/flow/server/communication/JavaScriptBootstrapHandler.java

@@ -28,7 +28,6 @@
 
 import com.vaadin.flow.component.PushConfiguration;
 import com.vaadin.flow.component.UI;
-import com.vaadin.flow.component.page.ExtendedClientDetails;
 import com.vaadin.flow.internal.BootstrapHandlerHelper;
 import com.vaadin.flow.internal.DevModeHandler;
 import com.vaadin.flow.internal.DevModeHandlerManager;
@@ -171,40 +170,9 @@ protected BootstrapContext createAndInitUI(Class<? extends UI> uiClass,
 
         config.put("requestURL", requestURL);
 
-        // Parse browser details from request parameters and store in UI
-        extractAndStoreBrowserDetails(request, context.getUI());
-
         return context;
     }
 
-    /**
-     * Extracts browser details from the request JSON parameter and stores them
-     * in the UI's internals as ExtendedClientDetails.
-     *
-     * @param request
-     *            the request containing browser details as JSON parameter
-     * @param ui
-     *            the UI instance to store the details in
-     */
-    private void extractAndStoreBrowserDetails(VaadinRequest request, UI ui) {
-        // Extract browser details JSON parameter from request
-        // This is sent by the client in the v-r=init request
-        String browserDetailsJson = request.getParameter("v-browserDetails");
-
-        if (browserDetailsJson != null && !browserDetailsJson.isEmpty()) {
-            try {
-                JsonNode json = JacksonUtils.readTree(browserDetailsJson);
-                ExtendedClientDetails details = ExtendedClientDetails
-                        .fromJson(ui, json);
-                ui.getInternals().setExtendedClientDetails(details);
-            } catch (Exception e) {
-                // Log and continue without browser details
-                getLogger().debug(
-                        "Failed to parse browser details from init request", e);
-            }
-        }
-    }
-
     @Override
     protected void initializeUIWithRouter(BootstrapContext context, UI ui) {
     }