An oversight - and a bad one too. I'll fix it as soon as I can get
reconnected to Apache's CVS.
Mike
On Thursday 07 June 2001 19:01, you wrote:
> "Stover, Michael" wrote:
> > > Maybe I'm not understanding something somewhere... maybe you
> > > can point me in the right direction?
> >
> > Latest CVS code, not yet released as a tar.
>
> ok, I just downloaded, compiled, and ran the latest
> CVS code, and I see that the protocol and port issue
> has been fixed, but I don't see any code in UrlConfig
> or HTTPSampler which handles query strings separate
> from paths in the url...
>
> just to be clear, I'm talking about
>
> protocol://host/path?querystring
>
> Are other people just constructing UrlConfig's
> with the query string appended to the path?
>
> Am I the only one passing arguments to servlets using GET?
>
> I'd propose something like this:
>
> Index: UrlConfig.java
> ===================================================================
> RCS file:
> /home/cvspublic/jakarta-jmeter/src/org/apache/jmeter/protocol/http/config/U
>rlConfig.java,v retrieving revision 1.9
> diff -r1.9 UrlConfig.java
> 108c108
> < return new URL(getProtocol(),getDomain(),getPath());
> ---
>
> > return new URL(getProtocol(),getDomain(),getPath() +
>getQueryString()
> > );
>
> 112c112
> < return new URL(getProtocol(), (String)properties.get(DOMAIN),
> getPort(), (String)properties.get(PATH));
> ---
>
> > return new URL(getProtocol(), (String)properties.get(DOMAIN),
> > getPort(), (String)properties.get(PATH) + getQueryString() );
>
> 177a178,192
>
> > }
> >
> > public void setQueryString(String qs) {
> > properties.put(QUERYSTRING, qs);
> > }
> >
> > public String getQueryString() {
> > String qs = (String) properties.get(QUERYSTRING);
> > if( qs == null || qs.equals("") ) {
> > return "";
> > }
> > if( ! qs.startsWith("?") ) {
> > qs = "?" + qs;
> > }
> > return qs;
>
> Peter
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
--
Mike Stover
[EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
