struts-faces bug

[Anand Joshi <aj...@hotmail.com>]

All,
I am using Struts-faces Integration library 1.0. I recently came across the 
problem using HttpServletRequestWrapper from it. It implements 
HttpServletRequest class but per spec. servlet 2-4, all request wrapppers 
should extend javax.servlet.http.HttpServletRequestWrapper class.
Here is sec. 8.2 of servlet 2.4 spec
To use a request dispatcher, a servlet calls either the include method or 
forward
method of the RequestDispatcher interface. The parameters to these methods 
can
be either the request and response arguments that were passed in via the 
service
method of the javax.servlet interface, or instances of subclasses of the 
request
and response wrapper classes that were introduced for version 2.3 of the
specification. In the latter case, the wrapper instances must wrap the 
request or
response objects that the container passed into the service method

It does not work when used with some popular appservers having 2.4 
webcontainer and I had to change it to extend HttpServletRequestWrapper to 
make it work.
I would like to know the intent of getServletPath() in this wrapper.

thanks,

Anand Joshi,

_________________________________________________________________
Express yourself instantly with MSN Messenger! Download today - it's FREE! 
hthttp://messenger.msn.click-url.com/go/onm00200471ave/direct/01/


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org

Re: struts-faces bug

[Craig McClanahan <cr...@gmail.com>]

Yah, that's definitely broken.  Could you please submit a bug report
to our issue tracking system?

http://cvs.apache.org/bugzilla/

As to the intent, this wrapper was proposed by the MyFaces folks --
it's not needed for the library to work with the JSF RI ... maybe one
of those guys can comment on why it's needed for MyFaces to work?

Craig


On Fri, 29 Oct 2004 15:39:56 -0400, Anand Joshi <aj...@hotmail.com> wrote:
> All,
> I am using Struts-faces Integration library 1.0. I recently came across the
> problem using HttpServletRequestWrapper from it. It implements
> HttpServletRequest class but per spec. servlet 2-4, all request wrapppers
> should extend javax.servlet.http.HttpServletRequestWrapper class.
> Here is sec. 8.2 of servlet 2.4 spec
> To use a request dispatcher, a servlet calls either the include method or
> forward
> method of the RequestDispatcher interface. The parameters to these methods
> can
> be either the request and response arguments that were passed in via the
> service
> method of the javax.servlet interface, or instances of subclasses of the
> request
> and response wrapper classes that were introduced for version 2.3 of the
> specification. In the latter case, the wrapper instances must wrap the
> request or
> response objects that the container passed into the service method
> 
> It does not work when used with some popular appservers having 2.4
> webcontainer and I had to change it to extend HttpServletRequestWrapper to
> make it work.
> I would like to know the intent of getServletPath() in this wrapper.
> 
> thanks,
> 
> Anand Joshi,
> 
> _________________________________________________________________
> Express yourself instantly with MSN Messenger! Download today - it's FREE!
> hthttp://messenger.msn.click-url.com/go/onm00200471ave/direct/01/
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
> 
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org