Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] SSL support for the mail proxy module

Kunal Pariani
September 16, 2014 03:32AM
Updated the diffs after addressing the first 2 issues.
Regarding the 3rd comment, you are correct. Only 1 set of ssl settings for all the mail backends with my patch. I guess this will be a limitation with the current mail proxy workflow ? Am not sure of the exact changes that will be required to address this issue completely.
Also skipped the non-trivial name related ssl directives from http proxy module as suggested by Maxim.

Thanks
-Kunal

----- Original Message -----
From: "Filipe Da Silva" <fdasilvayy@gmail.com>
To: "nginx-devel" <nginx-devel@nginx.org>
Sent: Monday, September 15, 2014 10:48:10 PM
Subject: Re: [PATCH] SSL support for the mail proxy module

Hello,

I see at least three issues in this patch:
- a mixed use of NGX_MAIL_SSL and NGX_HTTP_SSL defines
- a wrong patch format: please add this item to your .hgrc config file:
[diff]
showfunc = True
- as Maxim previously explains, there a conflict with the mail proxy workflow and your changes.
Currently, the mail back-end server address ( ip+port) is dynamically returned by the auth script ( called by HTTP ).
With your patch, there is only one set of ssl settings for all the mail back-ends, if I'm not wrong ?

HTH and regards,
FDS.

2014-09-14 1:14 GMT+02:00 Kunal Pariani <kpariani@zimbra.com>:
> # HG changeset patch
> # User Kunal Pariani <kpariani@zimbra.com>
> # Date 1410649455 18000
> # Sat Sep 13 18:04:15 2014 -0500
> # Node ID 5c2524403ab7c870b1fa7294f935d3292f25fd1d
> # Parent e3016ee8dba396614f28a3644996b8cc6de8f9e3
> SSL support for the mail proxy module
>
> diff -r e3016ee8dba3 -r 5c2524403ab7 src/mail/ngx_mail_proxy_module.c
> --- a/src/mail/ngx_mail_proxy_module.c Sat Sep 13 21:47:13 2014 +0400
> +++ b/src/mail/ngx_mail_proxy_module.c Sat Sep 13 18:04:15 2014 -0500

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] SSL support for the mail proxy module

Kunal Pariani 826 September 13, 2014 07:16PM

Re: [PATCH] SSL support for the mail proxy module

Filipe Da Silva 419 September 16, 2014 01:50AM

[PATCH] SSL support for the mail proxy module

Kunal Pariani 391 September 16, 2014 03:14AM

Re: [PATCH] SSL support for the mail proxy module

Kunal Pariani 529 September 16, 2014 03:32AM

Re: [PATCH] SSL support for the mail proxy module

Maxim Dounin 404 September 16, 2014 08:04AM

Re: [PATCH] SSL support for the mail proxy module

Kunal Pariani 398 September 16, 2014 08:24PM

Re: [PATCH] SSL support for the mail proxy module

Franck Levionnois 419 October 21, 2014 04:00AM

Re: [PATCH] SSL support for the mail proxy module

Franck Levionnois 448 November 13, 2014 11:12AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 141
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready