[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