Merge pull request Netflix#199 from ausmith/1.x

Handle query params without an `=`
sanjeevgour · Apr 6, 2016 · 70bae0b · 70bae0b
2 parents 80eb6da + 11ae37f
commit 70bae0b
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/zuul-simple-webapp/src/main/groovy/filters/route/SimpleHostRoutingFilter.groovy b/zuul-simple-webapp/src/main/groovy/filters/route/SimpleHostRoutingFilter.groovy
@@ -261,14 +261,15 @@ class SimpleHostRoutingFilter extends ZuulFilter {
             if (rebuiltQueryString.length() > 0) {
                 rebuiltQueryString = rebuiltQueryString + "&"
             }
-            def (name,value) = keyPair.split("=", 2)
-            if (value != null) {
+
+            if (keyPair.contains("=")) {
+                def (name,value) = keyPair.split("=", 2)
                 value = URLDecoder.decode(value, encoding)
                 value = new URI(null, null, null, value, null).toString().substring(1)
                 value = value.replaceAll('&', '%26')
                 rebuiltQueryString = rebuiltQueryString + name + "=" + value
             } else {
-                name = URLDecoder.decode(name, encoding)
+                def value = URLDecoder.decode(keyPair, encoding)
                 value = new URI(null, null, null, value, null).toString().substring(1)
                 rebuiltQueryString = rebuiltQueryString + value
             }