jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Milamber <milam...@apache.org>
Subject Re: svn commit: r1693815 - /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
Date Sun, 02 Aug 2015 14:12:52 GMT
Thanks Felix to fix up the issue!

On 02/08/2015 14:53, fschumacher@apache.org 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) {
>
>
>


Mime
View raw message