On Thursday 25 June 2015 14:51:15 Jon Nalley wrote:
> # HG changeset patch
> # User Jon Nalley <code@bluebot.org>
> # Date 1435261685 18000
> # Thu Jun 25 14:48:05 2015 -0500
> # Node ID 894a268769bf9b3c806a9506ea1e8daf2dac3841
> # Parent 6345822f0abb70807f635989b6c2df7852a55bd9
> Adds $orig_remote_addr in realip module
>
> When the realip module sets $remote_addr, the connecting IP is
> no longer available for logging etc. This change preserves the
> connecting IP as $orig_remote_addr.
>
> diff -r 6345822f0abb -r 894a268769bf src/core/ngx_connection.h
> --- a/src/core/ngx_connection.h Thu Jun 25 12:36:52 2015 +0300
> +++ b/src/core/ngx_connection.h Thu Jun 25 14:48:05 2015 -0500
> @@ -145,6 +145,10 @@
> socklen_t socklen;
> ngx_str_t addr_text;
>
> +#if (NGX_HTTP_REALIP)
> + ngx_str_t orig_addr_text;
> +#endif
> +
> ngx_str_t proxy_protocol_addr;
>

The realip module works on a per request basis, so storing this
information inside the connection structure looks wrong to me.

wbr, Valentin V. Bartenev

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel