You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Vincent Bray <no...@gmail.com> on 2007/08/01 03:47:04 UTC

Re: svn commit: r561477 - in /httpd/httpd/trunk/docs/manual/mod: mod_proxy.html.en mod_proxy.xml

On 01/08/07, slive@apache.org <sl...@apache.org> wrote:
> Author: slive
> Date: Tue Jul 31 13:13:50 2007
> New Revision: 561477
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=561477
> Log:
> Docuemnt the x- headers added by mod_proxy_http.
>
> PR: 41097
>
> Modified:
>     httpd/httpd/trunk/docs/manual/mod/mod_proxy.html.en
>     httpd/httpd/trunk/docs/manual/mod/mod_proxy.xml

> Modified: httpd/httpd/trunk/docs/manual/mod/mod_proxy.xml
> +      <dd>The original host requested by the client in the <code>Host</code>
> +       HTTP request ehader.</dd>

s/ehader/header/

Also, for consistency, should the argument to RewriteOptions in your
earlier patch regarding vhosts be capitalised like RewriteOptions
Inherit rather than RewriteOptions inherit? It's a small matter, but
these things get cargo-culted and lead to the current belief that
'require' should always be lowercase :(

-- 
noodl

Re: svn commit: r561477 - in /httpd/httpd/trunk/docs/manual/mod: mod_proxy.html.en mod_proxy.xml

Posted by Vincent Bray <no...@gmail.com>.
On 01/08/07, Vincent Bray <no...@gmail.com> wrote:
> s/ehader/header/

D'oh! Wrong. List. Sorry.

-- 
noodl