Grzegorz Kossakowski pisze: > Grzegorz Kossakowski pisze: >> Felix Knecht pisze: >> I can confirm now that I see this problem locally. By mistake, I was testing >> a copy without this bug >> fix before committing so I didn't spot it. >> >> I'm just having a look at this. >> >> Sorry for breaking your work at the end of the week! :-( > > I have looked at this and found the cause for this bug. Class > StatusRetrievableBufferedWrappedResponse passes Content-Length setting to the > wrapped response > _before_ final settlement of this value which leads to situation where > Content-Length is set to 0. > > I tried to fix this but whole issue looks to be more troublesome that I > previously thought and I > need to rethink whole approach. Basically, I opt to writing our own, > full-blown implementation of > HttpServletResponse that will buffer *everything* until we can be 100% sure > that everything is > established in final shape. > > This looks like quite a lot of work to implement meaningful implementation of > HttpServletResponse > that can copy to another one and will certainly increase complexity of SSF. > > If you have any thoughts on this I'm willing to hear.
I regret to say that I probably won't be able to fix this until final release of SSF. I reverted commit blowing up SSF but this does not solves the whole issue, of course. The original bug is still there and it will bite people even in more obscure way because its real consequences can be seen in complicated setups with servlet inheritance only. Nevertheless, it's still better than completely broken SSF. -- Grzegorz Kossakowski Committer and PMC Member of Apache Cocoon http://reflectingonthevicissitudes.wordpress.com/
