Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Maxim Dounin
November 10, 2016 10:02AM
Hello!

On Thu, Nov 10, 2016 at 01:06:54AM +0100, Bjørnar Ness wrote:

[...]

> > Current question is:
> >
> > What "listen ... proxy_protocol" should mean in case of mail. In
> > other modules, it just means that PROXY protocol header is parsed
> > and appropriate variables are available for use. It would be good
> > to have similar meaning in mail, but there are realip module and
> > no variables in mail.
>
> But Auth module can get the variables passed via headers, which is
> certainly a usecase, also, to be able to send same proxy protocol header out
> as you get in, the proxy-proxy-protocol scenario, it needs to be
> stored somewhere.
> This will work seemlessly on both mail, http and stream when proxy-protocol is
> enabled in both listen and outgoing, think of it as a "transparent
> smart-proxy" :)

It looks like then only use case you have in mind is nginx between
some frontend which adds a PROXY protocol header and a backend
which is able to accept such a header. Certainly this is not the
only real use case, but just one of multiple possible ones.

Other use cases include:

- nginx behind some balancer which adds PROXY protocol, and a
backend which doesn't understand PROXY protocol behind it;

- nginx in front of a backend which understands PROXY protocol,
and nothing in front of nginx.

Also, every time I see the word "smart" I start thinking about
security problems introduced along the way.

> > In the stream module similar problem was resolved by not
> > introducing "listen ... proxy_protocol" till variables support was
> > added, and by adding realip module at the same time. May be there
> > are better options.
> >
> > I certainly dislike what is currently suggested, that is, just
> > passing an address provided via PROXY protocol to backends via
> > XCLIENT.
> >
> > Introducing PROXY protocol to backends instead of XCLIENT looks
> > as a separate thing.
>
> I think adding more support for XCLIENT these days is not needed, as the
> software in common use today supports proxy protocol native.

This doesn't seem to take into account the fact that PROXY
protocol can only pass addresses, while XCLIENT is able to provide
various other information like a client login, hostname and so on.

--
Maxim Dounin
http://nginx.org/

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Bjørnar Ness 429 September 18, 2016 08:14AM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Bjørnar Ness 209 September 19, 2016 03:50PM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Dmitry Volyntsev 207 September 20, 2016 07:18AM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Bjørnar Ness 189 September 20, 2016 08:52AM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Dmitry Volyntsev 232 September 20, 2016 11:20AM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Bjørnar Ness 207 September 20, 2016 01:20PM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Bjørnar Ness 177 November 03, 2016 03:38PM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Maxim Dounin 155 November 09, 2016 01:22PM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Bjørnar Ness 179 November 09, 2016 02:54PM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Maxim Dounin 156 November 09, 2016 06:54PM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Bjørnar Ness 159 November 09, 2016 07:08PM

Re: [PATCH] proxy-protocol dst variables and proxy-proxy-protocol

Maxim Dounin 185 November 10, 2016 10:02AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 298
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready