- Fix API object filter to only add Querystring when Queryparameters are not empty

AlejandroP · AlejandroP · commit 6f9fb8039377 · 2023-11-19T23:36:00.000-03:00
diff --git a/java/src/main/java/com/genexus/filters/APIObjectFilter.java b/java/src/main/java/com/genexus/filters/APIObjectFilter.java
@@ -39,7 +39,11 @@ public  void doFilter(IServletRequest request, IServletResponse response, IFilte
 				}
             }     
             if(isPath) {
-                String fwdURI = REST_SUBPART  + path + QS_SEP + httpRequest.getQueryString();
+                String fwdURI = REST_SUBPART  + path;
+				String qString = httpRequest.getQueryString();
+				if ( qString != null && !qString.isEmpty()) {
+					fwdURI = fwdURI + QS_SEP + qString;
+				}
 				logger.info("Forwarding from " + path +" to: " + fwdURI) ;
 				httpRequest.getRequestDispatcher(fwdURI).forward(request,response);
             }

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,11 @@ public void doFilter(IServletRequest request, IServletResponse response, IFilte`
`39`	`39`	`}`
`40`	`40`	`}`
`41`	`41`	`if(isPath) {`
`42`		`- String fwdURI = REST_SUBPART + path + QS_SEP + httpRequest.getQueryString();`
	`42`	`+ String fwdURI = REST_SUBPART + path;`
	`43`	`+ String qString = httpRequest.getQueryString();`
	`44`	`+ if ( qString != null && !qString.isEmpty()) {`
	`45`	`+ fwdURI = fwdURI + QS_SEP + qString;`
	`46`	`+ }`
`43`	`47`	`logger.info("Forwarding from " + path +" to: " + fwdURI) ;`
`44`	`48`	`httpRequest.getRequestDispatcher(fwdURI).forward(request,response);`
`45`	`49`	`}`