Re: [PATCH] Add query string functionality testing for headers/

["Ilya A. Volynets-Evenbakh" <ilya@total-knowledge.com>]

Query string is not a header. This belongs elsewhere (probably completely
separate servlet for this one)

Sergey wrote:
> From: Sergey Jukov <sergey@total-knowledge.com>
>
> ---
>  ChangeLog           |    3 +++
>  headers/headers.cpp |    1 +
>  headers/headers.h   |    2 ++
>  3 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/ChangeLog b/ChangeLog
> index d3864e9..97fc664 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,6 @@
> +Sergey Jukov <sergey@total-knowledge>                Tue, 13 Sep 2006 13:00:00 -0800 
> +- Add query string test functionality to headers/headers.cpp and headers/header.h.
> +
>  Sergey Jukov <sergey@total-knowledge>                Tue, 13 Sep 2006 11:00:00 -0800 
>  - Fix copyright issue in headers/headers.cpp and headers/header.h.
>  	
> diff --git a/headers/headers.cpp b/headers/headers.cpp
> index cf247e2..6fb3002 100644
> --- a/headers/headers.cpp
> +++ b/headers/headers.cpp
> @@ -46,6 +46,7 @@ void HeadersServlet::service(servlet::Ht
>          s << l;
>  	std:: string dateHeader = s.str();
>  	out << "\n" << testHeaderName+" value = "+dateHeader; 
> +	out << "\nQuery String: "+req.getQueryString(); 
>  	out<<"</PRE>";
>  	renderFooter(out);
>  }
> diff --git a/headers/headers.h b/headers/headers.h
> index 3247dbf..9a807cf 100644
> --- a/headers/headers.h
> +++ b/headers/headers.h
> @@ -37,6 +37,8 @@ milliseconds since January 1, 1970 GMT, 
>  included with the request. Throws IllegalArgumentException if the header value
>  can't be converted to a date.
>  Example provided for If-Modified-Since HTTP request header using testHeaders.pl
> +It demonstrates work of getQueryString() function, if no query string provided,
> +returns null.
>  
>  @author Ilya A. Volynets-Evenbakh
>  */
>   

-- 
Ilya A. Volynets-Evenbakh
Total Knowledge. CTO
http://www.total-knowledge.com