Re: content-length (was cvs commit: xml-cocoon/src/org/apache/cocoon Engine.java

[Robin Green <gr...@hotmail.com>]

balld@locus.apache.org wrote:
>+                }
>+
>+                if (!VERBOSE && page.isText()) {
>+                    response.setContentLength(page.getContent().length());

Unfortunately that won't work - it needs to be in number of bytes, which 
isn't necessarily the same as the number of characters.

I have a version sitting on my disc that should work, but there are some 
other related changes I'm working on so I can't commit it just yet - I hope 
to be making an announcement on this soon...



_____________________________________________________________________________________
Get more from the Web.  FREE MSN Explorer download : http://explorer.msn.com

Re: content-length (was cvs commit: xml-cocoon/src/org/apache/cocoon Engine.java

[Donald Ball <ba...@webslingerZ.com>]

On Wed, 6 Dec 2000, Robin Green wrote:

> balld@locus.apache.org wrote:
> >+                }
> >+
> >+                if (!VERBOSE && page.isText()) {
> >+                    response.setContentLength(page.getContent().length());
>
> Unfortunately that won't work - it needs to be in number of bytes, which
> isn't necessarily the same as the number of characters.

er, well, i patched a quick fix that should work - and even do the
encoding stuff right - but feel free to change it if you want - just
thought i'd get a jump start. :)

- donald