Tags:
create new tag
, view all tags

Bug: Unwanted chars in included web page

INCLUDE{"http://m4l.berlios.de/cgi-bin/nflash-0.7.2/news.cgi"} shows additional chars, which aren't in the original page. We always get a "0" at the end of the include and somtimes some chars in the middle of the included file.

Test case

Compare: http://m4l.berlios.de/cgi-bin/nflash-0.7.2/news.cgi (source) and http://m4l.berlios.de/cgi-bin/twiki-2/bin/view/Main/WebHome You find a "0" at the end of the included page and the text "Commen 52b ts: 0" instead of "Comments: 0" in the fourth news box. (This error can be very ugly, if the additional chars destroy html tags like "table")

Environment

TWiki version: TWikiBetaRelease
TWiki plugins: DefaultPlugin, EmptyPlugin, InterwikiPlugin
Server OS: Solaris 9 (x86)
Web server: Apache/1.3.27 (Unix) mod_ssl/2.8.14 OpenSSL/0.9.6i PHP/4.3.2 PHP/3.0.18 mod_perl/1.25
Perl version: 5.6.1
Client OS: Linux version 2.4.20
Web Browser: Mozilla Firebird 0.6.1

-- RonnyZiegler - 15 Aug 2003

Follow up

This is due to TWiki claiming to origin servers to support HTTP/1.1 - which requires clients to support ChunkedTransferEncoding . TWiki uses a very simplistic implementation of an HTTP client and barely meets the HTTP/1.0 specification, which is why you are seeing this error.

Fix record

A fix which should (99.999% certain) solve your problem suitable for applying against BeijingRelease (not the latest beta) is available on the page ProxiedIncludesBrokenImplementationBug . This fix has been available since 09 Jul 2003, has been tested in production and is known good. (However I wasn't looking for ChunkedTransferEncoding problems - but then I should not have seen any since ChunkedTransferEncoding was introduced in HTTP/1.1 and did not exist in HTTP/1.0 )

At some point I'll port the patch to the latest beta.

-- MichaelSparks - 15 Aug 2003

Resolved record

I applied your proposed patch http://twiki.org/p/pub/Codev/ProxiedIncludesBrokenImplementationBug/PROXYBUG.patch and everything works fine now. Thanks alot!

You should include this patch into the CVS version.

-- RonnyZiegler - 15 Aug 2003

Edit | Attach | Watch | Print version | History: r3 < r2 < r1 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r3 - 2003-08-15 - RonnyZiegler
 
  • Learn about TWiki  
  • Download TWiki
This site is powered by the TWiki collaboration platform Powered by Perl Hosted by OICcam.com Ideas, requests, problems regarding TWiki? Send feedback. Ask community in the support forum.
Copyright © 1999-2017 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.