Re: Code updates

On November 13, 2002 09:56 pm, Miles Elam wrote:
> This is what I have done to proto_http.c:
>
> 1961,1963c1961,1973
> <             // "%d" req->total_file_len
> <             memcpy(curr, &req->etag, req->lendigits);
> <             curr += req->lendigits;
> ---
>
>  >             /* The ETag was generated with an uncompressed file's
>  >              *  info.  I've left the ETag as it is, but the size
>  >              *  needs to be from req->total_file_len.  In other
>  >              *  words, needs to be from the compressed image if
>  >              *  we're sending gzip versions.
>  >              *  - [email protected] -- 2002-11-13
>  >              */
>  >             if (req->content_gzipped)
>  >                 curr += sprintf(curr, "%Ld", req->total_file_len);
>  >             else {
>  >                 memcpy(curr, &req->etag, req->lendigits);
>  >                 curr += req->lendigits;
>  >             }

This patch works perfectly with Mozilla 1.1, and Konqueror 3.1, Netscape 4.1.

It does work perfectly with IE5.0.
It does not work well with IE5.5:
1.  I click on the HTML, IE5.5 opens a new window with the compressed data.
2. close compressed data window, try again
3. same as #1 - compressed data window opens
4. same as #2
5. it works.

On another IE5.5 machine the compressed data window only opens once, then it 
works.

My Win4Lin module doesn't work after I patched this (my fault) so I can't test 
IE 6.x. Anyone?

IE5.5 is the most used browser, of course. Perhaps we need to test the User 
Agent as well before we send the compressed data.

Can anyone else confirm this?