Skip to content

Commit a64f19e

Browse files
committed
use raw query in hollow ui http handler
Previously, parsing post data from the request URI query string would decode sequences twice, first in `getQuery()`, then in `parseQueryString`. This would cause `'+'` characters to be converted to whitespace. Fix this by using `getRawQuery()` instead.
1 parent 1fd1a84 commit a64f19e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

hollow-ui-tools/src/main/java/com/netflix/hollow/ui/HttpHandlerWithServletSupport.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -305,7 +305,7 @@ public void setReadListener(ReadListener readListener) {
305305
Map<String, String[]> parsePostData = new HashMap<>();
306306

307307
try {
308-
parsePostData.putAll(HttpUtils.parseQueryString(ex.getRequestURI().getQuery()));
308+
parsePostData.putAll(HttpUtils.parseQueryString(ex.getRequestURI().getRawQuery()));
309309

310310
// check if any postdata to parse
311311
parsePostData.putAll(HttpUtils.parsePostData(inBytes.length, is));

0 commit comments

Comments
 (0)