Thanks Felix, nice fix On Sunday, August 2, 2015, Milamber <[email protected]> wrote:
> 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) { >> >> >> >> > -- Cordialement. Philippe Mouawad.
