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 <[EMAIL PROTECTED]> 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: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
