Re: [PATCH] Add proxy_protocol option to mail listener

2017-08-08 Thread Maxim Dounin
Hello! On Mon, Aug 07, 2017 at 03:23:12PM +0200, Kees Bos wrote: > # HG changeset patch > # User Kees Bos > # Date 1500565189 0 > #  Thu Jul 20 15:39:49 2017 + > # Node ID 327f18e079b175b14277a23e75715f5feee34d69 > # Parent  

Re: [PATCH] Add proxy_protocol option to mail listener

2017-08-08 Thread Wayde Nie
Thanks Kees and Maxim, for the patch and the discussion. A very helpful community! This looks like it's working well for my use case for a tcp465/"ssl on" mail proxy server block. My earlier issues appear to have (at least partly) been related to STARTTLS with a tcp587/"starttls only" block that

Re: [PATCH] Add proxy_protocol option to mail listener

2017-08-07 Thread Kees Bos
On ma, 2017-08-07 at 07:49 -0400, Wayde Nie wrote: > > > proxy_protocol > > > on the listen directive I see the correct ip and port picked up > > > and > > > logged in the error.log, however, then nginx stops sending the > > > smtp > > > greeting... My mail client connects to my loadbalancer, the

Re: [PATCH] Add proxy_protocol option to mail listener

2017-08-07 Thread Wayde Nie
On 08/07/2017 03:44 AM, Kees Bos wrote:, and when I do set >> proxy_protocol >> on the listen directive I see the correct ip and port picked up and >> logged in the error.log, however, then nginx stops sending the smtp >> greeting... My mail client connects to my loadbalancer, the lb >> connects

Re: [PATCH] Add proxy_protocol option to mail listener

2017-08-07 Thread Kees Bos
On za, 2017-08-05 at 20:44 -0400, Wayde Nie wrote: > Hi Kees, Maxim, > > I'm very interested in proxy_protocol enabled nginx mail proxy as > well. > Wondering if the feature might be more straightforward (initially?) > if, > when enabled on the listen line, it simply set the appropriate >

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-20 Thread Kees Bos
___ nginx-devel mailing list nginx-devel@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-devel

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-20 Thread Maxim Dounin
Hello! On Thu, Jul 20, 2017 at 09:52:08AM +0200, Kees Bos wrote: > On di, 2017-07-18 at 18:02 +0300, Maxim Dounin wrote: > > Hello! > > > > On Tue, Jul 18, 2017 at 03:13:21PM +0200, Kees Bos wrote: > > > > > > > > Some inline stuff just to be sure I do understand what you mean. > > > > > >

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-20 Thread Kees Bos
On di, 2017-07-18 at 18:02 +0300, Maxim Dounin wrote: > Hello! > > On Tue, Jul 18, 2017 at 03:13:21PM +0200, Kees Bos wrote: > > > > > Some inline stuff just to be sure I do understand what you mean. > > > > On di, 2017-07-18 at 15:56 +0300, Maxim Dounin wrote: > > > > > > Hello! > > > > > >

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-18 Thread Kees Bos
Some inline stuff just to be sure I do understand what you mean. On di, 2017-07-18 at 15:56 +0300, Maxim Dounin wrote: > Hello! > > On Tue, Jul 18, 2017 at 12:06:09PM +0200, Kees Bos wrote: > > > > > # HG changeset patch > > # User Kees Bos > > # Date 1500371531 0 > >

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-18 Thread Maxim Dounin
Hello! On Tue, Jul 18, 2017 at 12:06:09PM +0200, Kees Bos wrote: > # HG changeset patch > # User Kees Bos > # Date 1500371531 0 > #  Tue Jul 18 09:52:11 2017 + > # Node ID 8dd6050ca6858d9bea139067611ca5c69cfe8f18 > # Parent  

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-18 Thread Kees Bos
# HG changeset patch # User Kees Bos # Date 1500371531 0 #  Tue Jul 18 09:52:11 2017 + # Node ID 8dd6050ca6858d9bea139067611ca5c69cfe8f18 # Parent  e3723f2a11b7ec1c196d59c331739bc21d9d9afd Add proxy_protocol option to mail listener Add support for the mail

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-12 Thread Kees Bos
On wo, 2017-07-12 at 15:56 +0300, Maxim Dounin wrote: > On Wed, Jul 12, 2017 at 02:08:31PM +0200, Kees Bos wrote: > On di, 2017-07-11 at 18:12 +0300, Maxim Dounin wrote: > > > On Fri, Jul 07, 2017 at 03:38:02PM +0200, Kees Bos wrote: > > > 2. It unconditionally trusts all clients who can connect

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-12 Thread Maxim Dounin
Hello! On Wed, Jul 12, 2017 at 02:08:31PM +0200, Kees Bos wrote: > On di, 2017-07-11 at 18:12 +0300, Maxim Dounin wrote: > > Hello! > > > > On Fri, Jul 07, 2017 at 03:38:02PM +0200, Kees Bos wrote: > > > > > > > > # HG changeset patch > > > # User Kees Bos > > > #

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-12 Thread Kees Bos
On di, 2017-07-11 at 18:12 +0300, Maxim Dounin wrote: > Hello! > > On Fri, Jul 07, 2017 at 03:38:02PM +0200, Kees Bos wrote: > > > > > # HG changeset patch > > # User Kees Bos > > # Date 1499422505 0 > > #  Fri Jul 07 10:15:05 2017 + > > # Node ID

Re: [PATCH] Add proxy_protocol option to mail listener

2017-07-11 Thread Maxim Dounin
Hello! On Fri, Jul 07, 2017 at 03:38:02PM +0200, Kees Bos wrote: > # HG changeset patch > # User Kees Bos > # Date 1499422505 0 > #  Fri Jul 07 10:15:05 2017 + > # Node ID bc79b2baf494aabb889de1e5dbe3184ff0cb9bfa > # Parent