Hello!

On Thu, Nov 03, 2016 at 08:37:04PM +0100, Bjørnar Ness wrote:

> Maxim: what needs change to get this merged? Followup will be mail pp
> support, which I saw a patch for today from somone else.

If this was a question to me, then the answer is:

I'm not convinced this should be merged. For the use case of
PROXY protocol bypass a more logical solution would be to avoid
removing the PROXY protocol header at all. If one needs to bypass
it and extract the information at the same time, something similar
to ssl preread as recently implemented in the stream module might
be a better solution.

If you have some ideas on how to properly support PROXY protocol
in mail - please comment on the relevant patch.

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

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