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

Maxim Dounin mdounin at mdounin.ru
Wed Nov 9 18:20:48 UTC 2016


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/



More information about the nginx-devel mailing list