Thanks Felix to fix up the issue!

On 02/08/2015 14:53, [email protected] wrote:
Author: fschumacher
Date: Sun Aug  2 13:53:18 2015
New Revision: 1693815

URL: http://svn.apache.org/r1693815
Log:
Bug 58137: Warn about urls that had to be escaped.
Bugzilla Id: 58137

Modified:
     
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java

Modified: 
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java?rev=1693815&r1=1693814&r2=1693815&view=diff
==============================================================================
--- 
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
 (original)
+++ 
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
 Sun Aug  2 13:53:18 2015
@@ -1361,8 +1361,9 @@ public abstract class HTTPSamplerBase ex
          }
          try {
              String escapedUrl = 
ConversionUtils.escapeIllegalURLCharacters(url);
-            if(log.isDebugEnabled()) {
-                log.debug("Successfully escaped url:'"+url +"' 
to:'"+escapedUrl+"'");
+            if (!escapedUrl.equals(url)) {
+                log.warn("Url '" + url + "' has been escaped to '" + escapedUrl
+                        + "'. Please corect your webpage.");
              }
              return escapedUrl;
          } catch (Exception e1) {




Reply via email to