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