Improved: Prevent URL parameters manipulation (OFBIZ-13147)

Allows JavaScriptEnabled=Y to pass.
apache · Oct 23, 2024 · 83f35c9 · 83f35c9
1 parent 612ad73
commit 83f35c9
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/framework/webapp/src/main/java/org/apache/ofbiz/webapp/control/ControlFilter.java b/framework/webapp/src/main/java/org/apache/ofbiz/webapp/control/ControlFilter.java
@@ -170,7 +170,9 @@ public void doFilter(HttpServletRequest req, HttpServletResponse resp, FilterCha
             String queryString = req.getQueryString();
             if (queryString != null) {
                 queryString = URLDecoder.decode(queryString, "UTF-8");
-                if (UtilValidate.isUrl(queryString) || !SecuredUpload.isValidText(queryString, Collections.emptyList())) {
+                if (UtilValidate.isUrl(queryString)
+                        || (!SecuredUpload.isValidText(queryString, Collections.emptyList())
+                                && !queryString.contains("JavaScriptEnabled=Y"))) {
                     Debug.logError("For security reason this URL is not accepted", MODULE);
                     throw new RuntimeException("For security reason this URL is not accepted");
                 }