feat: Make RequestUtil#getUrlMapping public for manual security configuration (#22446)

Copilot · mshabarov · tepi · web-flow · commit 7e43be2c19c3 · 2025-10-08T09:39:16.000+03:00
* Initial plan

* Make RequestUtil#getUrlMapping public method with documentation and tests

Co-authored-by: mshabarov &lt;61410877+mshabarov@users.noreply.github.com&gt;

---------

Co-authored-by: copilot-swe-agent[bot] &lt;198982749+Copilot@users.noreply.github.com&gt;
Co-authored-by: mshabarov &lt;61410877+mshabarov@users.noreply.github.com&gt;
Co-authored-by: Teppo Kurki &lt;teppo.kurki@vaadin.com&gt;
diff --git a/vaadin-spring/src/main/java/com/vaadin/flow/spring/security/RequestUtil.java b/vaadin-spring/src/main/java/com/vaadin/flow/spring/security/RequestUtil.java
@@ -393,7 +393,13 @@ private String getRequestRoutePath(HttpServletRequest request) {
                 .orElse(null);
     }
 
-    String getUrlMapping() {
+    /**
+     * Gets the url mapping for the Vaadin servlet.
+     *
+     * @return the url mapping
+     * @see VaadinConfigurationProperties#getUrlMapping()
+     */
+    public String getUrlMapping() {
         return configurationProperties.getUrlMapping();
     }
 
diff --git a/vaadin-spring/src/test/java/com/vaadin/flow/spring/security/RequestUtilTest.java b/vaadin-spring/src/test/java/com/vaadin/flow/spring/security/RequestUtilTest.java
@@ -656,4 +656,25 @@ static MockHttpServletRequest createRequest(String pathInfo,
         return r;
     }
 
+    @Test
+    public void testGetUrlMapping_fooMappedServlet_returnsMappingValue() {
+        Mockito.when(vaadinConfigurationProperties.getUrlMapping())
+                .thenReturn("/foo/*");
+        Assert.assertEquals("/foo/*", requestUtil.getUrlMapping());
+    }
+
+    @Test
+    public void testGetUrlMapping_rootMappedServlet_returnsMappingValue() {
+        Mockito.when(vaadinConfigurationProperties.getUrlMapping())
+                .thenReturn("/*");
+        Assert.assertEquals("/*", requestUtil.getUrlMapping());
+    }
+
+    @Test
+    public void testGetUrlMapping_nullMapping_returnsNull() {
+        Mockito.when(vaadinConfigurationProperties.getUrlMapping())
+                .thenReturn(null);
+        Assert.assertNull(requestUtil.getUrlMapping());
+    }
+
 }

Original file line number	Diff line number	Diff line change
`@@ -393,7 +393,13 @@ private String getRequestRoutePath(HttpServletRequest request) {`
`393`	`393`	`.orElse(null);`
`394`	`394`	`}`
`395`	`395`
`396`		`- String getUrlMapping() {`
	`396`	`+ /**`
	`397`	`+ * Gets the url mapping for the Vaadin servlet.`
	`398`	`+ *`
	`399`	`+ * @return the url mapping`
	`400`	`+ * @see VaadinConfigurationProperties#getUrlMapping()`
	`401`	`+ */`
	`402`	`+ public String getUrlMapping() {`
`397`	`403`	`return configurationProperties.getUrlMapping();`
`398`	`404`	`}`
`399`	`405`