From simon.urbanek at r-project.org  Wed Sep 30 15:33:53 2009
From: simon.urbanek at r-project.org (Simon Urbanek)
Date: Wed, 30 Sep 2009 09:33:53 -0400
Subject: [Rd] Problems connecting to httpd help server with some browsers
In-Reply-To: <4AC289B5.6070706@vanderbilt.edu>
References: <200909292115.04472.thomas.friedrichsmeier@ruhr-uni-bochum.de>
	<E73447DA-5192-4FE9-859F-E90ECC9C3B65@r-project.org>
	<4AC289B5.6070706@vanderbilt.edu>
Message-ID: <793369FE-802C-40E0-8493-5626ECEA1EF7@r-project.org>

Thanks, Jeff, good catch. Now committed.
Simon

On Sep 29, 2009, at 18:27 , Jeff Horner wrote:

> Simon Urbanek wrote:
> [...]
>> I don't have konqueror at hand, so I have tested and fixed the lynx  
>> case (lynx is acting as an HTTP/1.0 client and expects non- 
>> persistent connection). With some luck the fix may solve the  
>> konqueror issue as well (although I'd be a bit surprised if  
>> konqueror was not 1.1-capable..).
>
> Konqueror didn't like the extra newline in the Location header:
>
> Index: src/library/tools/R/dynamicHelp.R
> ===================================================================
> --- src/library/tools/R/dynamicHelp.R   (revision 49890)
> +++ src/library/tools/R/dynamicHelp.R   (working copy)
> @@ -172,7 +172,7 @@
>            return(list(payload = paste('Redirect to <a href="',  
> file, '">"',
>                         basename(file), '"</a>', sep=''),
>                        "content-type" = 'text/html',
> -                       header = paste('Location: ', file, '\n',  
> sep=''),
> +                       header = paste('Location: ', file, sep=''),
>                        "status code" = 302L)) # temporary redirect
>        } else if (length(file) > 1L) {
>             paths <- dirname(dirname(file))
>
>
> Jeff
>
>



[Rd] Problems connecting to httpd help server with some browsers
Simon Urbanek