httpd-docs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Luca Toscano <toscano.l...@gmail.com>
Subject Re: mod_remoteip.xml
Date Thu, 27 Apr 2017 08:57:56 GMT
2017-04-23 17:16 GMT+02:00 Lucien Gentis <lucien.gentis@univ-lorraine.fr>:

>
>
> Le 14/04/2017 à 15:26, Lucien Gentis a écrit :
>
> Hello Luca,
>
> Apologize for not answering.
>
> I come back next week
>
> Lucien
>
> Le 10/04/2017 à 19:20, Luca Toscano a écrit :
>
> Hello Lucien,
>
> 2017-04-08 17:28 GMT+02:00 Lucien Gentis <lucien.gentis@univ-lorraine.fr>:
>
>> Hello,
>>
>> Trunk branch : mod_remoteip.xml file, <name>RemoteIPProxyProtocolExceptions</name>
>> directive :
>>
>> "<usage>
>>      <p>The <directive>RemoteIPProxyProtocol</directive> directive
>> enables or"
>>
>> must probably be replaced by :
>>
>> "<usage>
>>      <p>The <directive>RemoteIPProxyProtocolExceptions</directive>
>> directive enables or"
>>
>
>
> I checked the last pending doc changes and the complete diff is:
>
> +    <p>The <code class="directive">RemoteIPProxyProtocol</code>
> directive enables or
> +    disables the reading and handling of the PROXY protocol connection
> header.
> +    Sometimes it is desirable to require clients to provide the PROXY
> header, but
> +    permit other clients to connect without it. This directive allows a
> server
> +    administrator to configure a single host or CIDR range of hosts that
> may do
> +    so. This is generally useful for monitoring and administrative
> traffic to a
> +    virtual host direct to the server behind the upstream load
> balancer.</p>
> +
>
> It may need to be explained a bit better but IIUC the paragraph is saying
> that since RemoteIPProxyProtocol is a on/off switch,
> RemoteIPProxyProtocolExceptions can be used to fine tune exceptions.
>
>
> OK, it's actually referencing RemoteIPProxyProtocol directive.
>
>
>
>> Same file : I'm not quite sure to understand the phrase "This is
>> generally useful for monitoring and administrative
>>     traffic to a virtual host direct to the server behind the upstream
>> load
>>     balancer."
>>
>
> Definitely not clear, need a bit of rework. Do you have something in mind
> to fix it?
>
>
> As far as I understand it, this directive allows to know client real IP
> address, and so better analyse a connection stream ; so, this phrase could
> be : "This is generally useful for monitoring network stream between
> clients and backend servers situated behind a load balancer"
>

After reading
https://httpd.apache.org/docs/trunk/mod/mod_remoteip.html#remoteipproxyprotocolexceptions
again I think that the sentence should be removed, since it is well
explained by what comes before it :)

Luca

Mime
View raw message