id="RemoteIPProxyProtocol">RemoteIPProxyProtocol name="remoteipproxyprotocol" id="remoteipproxyprotocol">Directive

> > +Syntax:

ProxyProtocol
> On|Optional|Off

In the above snippets I can see three different names: RemoteIPProxyProtocolEnable, RemoteIPProxyProtocol and ProxyProtocol. The new directive in the C code is called RemoteIPProxyProtocolEnable, but I can see new logs using also RemoteIPProxyProtocol (like "RemoteIPProxyProtocol: internal error: have data left over; "). I personally like a lot RemoteIPProxyProtocol (rather than RemoteIPProxyProtocolEnable that seems a bit heavy to read), but everything is fine as long as we use a single name, especially in the logs that admins will read :) I haven't checked the code in detail so I might say something completely irrelevant, just writing the first things that I noticed! Luca --001a114b1f6c781c280544e4a1b9 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

Hi Daniel and Jim,

I saw your comments = in one of the last email thread about Daniel's new code change for mod_= remoteip, and I have some questions for the doc about the naming of the new= directive:

= 2016-12-30 19:20 GMT+01:00 <elukey@apache.org>:
Author: elukey
Date: Fri Dec 30 18:20:04 2016
New Revision: 1776616

URL: http://svn.apache.org/viewvc?rev= =3D1776616&view=3Drev
Log:
Documentation rebuild for mod_remoteip

Modified:
=C2=A0 =C2=A0 httpd/httpd/trunk/docs/manual/mod/mod_remoteip.html.en =C2=A0 =C2=A0 httpd/httpd/trunk/docs/manual/mod/mod_remoteip.xml.fr
=C2=A0 =C2=A0 httpd/httpd/trunk/docs/manual/mod/mod_remoteip.xml.meta<= br>
+=C2=A0 =C2=A0 using the <code class=3D"directive"><a hr= ef=3D"#remoteipproxyprotocolenable">RemoteIPProxyPro= tocolEnable</a>

=C2=A0

+<div class=3D"directive-section"><h2><a name= =3D"RemoteIPProxyProtocol" id=3D"RemoteIPProxyProtocol"= >RemoteIPProxyProtocol</a> <a name=3D"remoteipproxypr= otocol" id=3D"remoteipproxyprotocol">Directive</a= ></h2>

=C2=A0

+<tr><th><a href=3D"directive-dict.html#Syntax&quo= t;>Syntax:</a></th><td><code>ProxyProtocol = On|Optional|Off</code></td></tr>=C2=A0

<= div>

=C2=A0

In the above snippets I can see three = different names: RemoteIPProxyProtocolEnable, RemoteIPProxyProtocol and Pro= xyProtocol. The new directive in the C code is called RemoteIPProxyProtocol= Enable, but I can see new logs using also RemoteIPProxyProtocol (like "= ;RemoteIPProxyProtocol: internal error: have data left over; ").=C2=A0=

I per= sonally like a lot RemoteIPProxyProtocol (rather than RemoteIPProxyProtocol= Enable that seems a bit heavy to read), but everything is fine as long as w= e use a single name, especially in the logs that admins will read :)

<= div class=3D"gmail_extra">