milamberspace commented on code in PR #5812:
URL: https://github.com/apache/jmeter/pull/5812#discussion_r1174592592


##########
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:
   @FSchumacher I not sure that I understand... the if condition with 
validNameAndValue is for put in the map name / know values with avoid to do 
this if the name&value is null (special case "&=&").  so the if condition must 
be just above the "map.put(name, known);" in line 326. so I don't understand 
how put the if condition more higher in the code?



-- 
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

Reply via email to