jmeter-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fschumac...@apache.org
Subject svn commit: r1693815 - /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
Date Sun, 02 Aug 2015 13:53:19 GMT
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) {



Mime
View raw message