FSchumacher commented on code in PR #5812:
URL: https://github.com/apache/jmeter/pull/5812#discussion_r1174599771
##########
src/protocol/http/src/main/java/org/apache/jmeter/protocol/http/visualizers/RequestViewHTTP.java:
##########
@@ -316,9 +322,10 @@ public static Map<String, String[]> getQueryMap(String
query) {
System.arraycopy(known, 0, tmp, 0, known.length);
known = tmp;
}
- map.put(name, known);
+ if (validNameAndValue) {
Review Comment:
You can take a look at
https://github.com/FSchumacher/jmeter/commit/02f8ce8c66c3434b5e8e565dff3c7835aabb75c1
where I implemented the changes. The tests pass.
In case of `=value` we have `paramSplit.length==2`and thus `name==""` and
`validNameAndValue==true`. That means, the code block inside the if condition
will still be run. (Or alternatively the `continue` will work)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@jmeter.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
