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
