You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2012/01/25 17:11:00 UTC
svn commit: r1235804 - in /jmeter/trunk:
src/protocol/http/org/apache/jmeter/protocol/http/visualizers/RequestViewHTTP.java
xdocs/changes.xml
Author: sebb
Date: Wed Jan 25 16:11:00 2012
New Revision: 1235804
URL: http://svn.apache.org/viewvc?rev=1235804&view=rev
Log:
Bug 52491 - Incorrect parsing of Post data parameters in Tree Listener / Http Request view
Modified:
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/visualizers/RequestViewHTTP.java
jmeter/trunk/xdocs/changes.xml
Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/visualizers/RequestViewHTTP.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/visualizers/RequestViewHTTP.java?rev=1235804&r1=1235803&r2=1235804&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/visualizers/RequestViewHTTP.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/visualizers/RequestViewHTTP.java Wed Jan 25 16:11:00 2012
@@ -234,6 +234,12 @@ public class RequestViewHTTP implements
String[] params = query.split(PARAM_CONCATENATE);
for (String param : params) {
String[] paramSplit = param.split("="); // $NON-NLS-1$
+ if (paramSplit.length > 2 ) {// detected invalid syntax (Bug 52491)
+ // Return as for SOAP above
+ map.clear();
+ map.put(" ", query); //blank name // $NON-NLS-1$
+ return map;
+ }
String name = null;
if (paramSplit.length > 0) {
name = paramSplit[0];
Modified: jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1235804&r1=1235803&r2=1235804&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml (original)
+++ jmeter/trunk/xdocs/changes.xml Wed Jan 25 16:11:00 2012
@@ -306,6 +306,7 @@ fix code which assumes that a new instan
<h3>Listeners</h3>
<ul>
<li>Bug 52357 - View results in Table does not allow for multiple result samples</li>
+<li>Bug 52491 - Incorrect parsing of Post data parameters in Tree Listener / Http Request view</li>
</ul>
<h3>Assertions</h3>