Support of IMAP ID command in mail proxy module?

2013-12-31 Thread Michael Kliewe
Hello, I'm using the mail module of nginx to proxy and loadbalance IMAP+POP3 connections to backend servers. I do authentication via http authentication. Some Clients are sending IMAP ID commands to the server with information about their software and version. I would like to log that and may

Re: [PATCH] Mail: send starttls flag value to auth script

2014-03-06 Thread Michael Kliewe
Hi Maxim, On Mar 6, 2014, at 5:27 PM, Maxim Dounin wrote: > Hello! > > On Thu, Mar 06, 2014 at 10:59:29AM +0100, Filipe da Silva wrote: > >> # HG changeset patch >> # User Filipe da Silva >> # Date 1394099468 -3600 >> # Thu Mar 06 10:51:08 2014 +0100 >> # Node ID 51fd90f96449c23af0076a19e

Re: [PATCH] Mail: send starttls flag value to auth script

2014-07-30 Thread Michael Kliewe
"Auth-Secured: ", + sizeof("Auth-Secured: ") - 1); +*b->last++ = s->connection->ssl ? '1' : '0' ; +*b->last++ = CR; *b->last++ = LF; +#endif + b->last = ngx_sprintf(b->last, "Auth-Login-Attempt: %ui&

Re: [PATCH] Mail: send starttls flag value to auth script

2014-08-01 Thread Michael Kliewe
Hi, you can rename it as you wish, as long as the functionality gets into nginx ;-) Michael Am 01.08.2014 um 20:59 schrieb Maxim Dounin: Hello! On Thu, Jul 31, 2014 at 04:32:16AM +0200, Michael Kliewe wrote: Hi Maxim, this is very interesting to have in nginx, and it's a very easy

Re: IMAP ID support in the mail proxy for NGINX

2014-09-18 Thread Michael Kliewe
Hi, Am 18.09.2014 um 19:59 schrieb Kunal Pariani: Zimbra has a patch implementing this IMAP ID command (http://www.rfc-base.org/txt/rfc-2971.txt). Would anyone from the nginx team comment on if they would like this from us ? Thanks -Kunal Please see the following patch, which Filipe implement

Re: [PATCH] Mail: send starttls flag value to auth script

2015-02-04 Thread Michael Kliewe
you could please merge it into nginx. Thanks! Michael Am 01.08.2014 um 22:14 schrieb Michael Kliewe: Hi, you can rename it as you wish, as long as the functionality gets into nginx ;-) Michael Am 01.08.2014 um 20:59 schrieb Maxim Dounin: Hello! On Thu, Jul 31, 2014 at 04:32:16AM +0200

Re: [PATCH] Mail: send starttls flag value to auth script

2015-02-25 Thread Michael Kliewe
, 2015 at 10:07:50PM +0100, Michael Kliewe wrote: Hi Maxim, I would like to remind again this feature patch. It would help a lot to get this information about transport encryption into the auth script. It does not hurt the performance, and is a very tiny patch. You can rename the header name and

Re: [PATCH] Mail: send starttls flag value to auth script

2015-03-02 Thread Michael Kliewe
t;nginx doesn't have the feature" (because of mixed nginx versions). With the original version of the patch this was possible. Kind regards Michael On Feb 25, 2015, at 4:31 PM, Michael Kliewe wrote: > Hi Maxim, > > thank you very much, that helps a lot! Then we can use th

Re: [PATCH] Mail: send starttls flag value to auth script

2015-03-02 Thread Michael Kliewe
Hi Maxim, On Mar 2, 2015, at 3:14 PM, Maxim Dounin wrote: > Hello! > > On Mon, Mar 02, 2015 at 01:12:44PM +0100, Michael Kliewe wrote: > >> with your changes there is a problem: >> nginx now just sends the header if the connection is encrypted. >> If the con

Re: [PATCH] Mail: send starttls flag value to auth script

2015-03-03 Thread Michael Kliewe
Hi again, On Mar 2, 2015, at 3:56 PM, Maxim Dounin wrote: > Hello! > > On Mon, Mar 02, 2015 at 03:32:03PM +0100, Michael Kliewe wrote: > >> Hi Maxim, >> >> On Mar 2, 2015, at 3:14 PM, Maxim Dounin wrote: >> >>> Hello! >>> >&g

Re: [PATCH] Mail: send starttls flag value to auth script

2015-03-03 Thread Michael Kliewe
Hi Maxim, On Mar 3, 2015, at 4:50 PM, Maxim Dounin wrote: > Hello! > > On Tue, Mar 03, 2015 at 03:14:50PM +0100, Michael Kliewe wrote: > >> Hi again, >> >> On Mar 2, 2015, at 3:56 PM, Maxim Dounin wrote: >> >> I'm sorry, I don't really wa

Re: [PATCH] Mail: add the "reuseport" option of the "listen" directive

2021-08-18 Thread Michael Kliewe
Am 18.08.2021 um 03:14 schrieb Robert Mueller: First, thanks for the patch. While the reuseport could cure (or hide if you will) the unbalancing you see it makes sense to get better understanding what exactly is going on. So far we haven't seen such weird behaviour ourself neither received re