Skip to content

Commit 5b27ead

Browse files
authored
Merge pull request #98 from mendix/backport/fix/xss
Fix for XSS vunerability in rest page footer
2 parents 67ae53f + d5baf63 commit 5b27ead

File tree

6 files changed

+5
-2
lines changed

6 files changed

+5
-2
lines changed

DIST/RestServices_mx7_4.2.3.mpk

852 KB
Binary file not shown.

RestServices.mpr

3 KB
Binary file not shown.

javasource/restservices/RestServices.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ public class RestServices {
2020
/**
2121
* Version of the RestServices module
2222
*/
23-
public static final String VERSION = "4.2.2";
23+
public static final String VERSION = "4.2.3";
2424

2525
/**
2626
* Amount of objects that are processed by the module at the same time.

javasource/restservices/util/Utils.java

+4-1
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@
2727
import com.mendix.systemwideinterfaces.core.meta.IMetaPrimitive;
2828
import com.mendix.systemwideinterfaces.core.meta.IMetaPrimitive.PrimitiveType;
2929

30+
import org.owasp.encoder.Encode;
31+
3032
public class Utils {
3133

3234
public static String getShortMemberName(String memberName) {
@@ -179,7 +181,8 @@ public static String nullToEmpty(String statusText) {
179181
}
180182

181183
public static String getRequestUrl(HttpServletRequest request) {
182-
return request.getRequestURL().toString() + (Utils.isEmpty(request.getQueryString()) ? "" : "?" + request.getQueryString());
184+
String queryString = Encode.forUriComponent(request.getQueryString());
185+
return request.getRequestURL().toString() + (Utils.isEmpty(queryString) ? "" : "?" + queryString);
183186
}
184187

185188
public static boolean isSystemAttribute(String key) {

userlib/encoder-1.2.1.jar

35.8 KB
Binary file not shown.

userlib/encoder-1.2.1.jar.RestServices.RequiredLib

Whitespace-only changes.

0 commit comments

Comments
 (0)