<div dir="ltr"><div><div>Hi, Here is the fixed version.<br></div>Rgds.<br></div>Filipe DA SILVA<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/5/20 Maxim Dounin <span dir="ltr"><<a href="mailto:mdounin@mdounin.ru" target="_blank">mdounin@mdounin.ru</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello!<br>
<div><div class="h5"><br>
On Mon, May 20, 2013 at 06:40:14PM +0200, Filipe Da Silva wrote:<br>
<br>
> Hello ,<br>
><br>
> Please find attach a two lines patch about ecdh_curve setting in<br>
> mail_ssl_module.<br>
><br>
> This setting is not applied, although it is declared and parsed from<br>
> nginx.conf file<br>
><br>
> I see this lack in 1.2.0 version too.<br>
<br>
</div></div>Nice catch, thanks.<br>
<br>
><br>
> Rgds,<br>
> Filipe DA SILVA<br>
<br>
> # HG changeset patch<br>
> # User fds <<a href="mailto:fdasilvayy@gmail.com">fdasilvayy@gmail.com</a>><br>
<br>
Could you please use a proper name here?  Just "fds" looks wierd.<br>
<br>
> # Date 1368197625 -7200<br>
> # Node ID f6c3e219648f47eaa65ab43a8189af31f7babc69<br>
> # Parent  7ffe7f3be4045dbc7478248370b473271660e473<br>
> Missing call to ngx_ssl_ecdh_curve in mail_ssl_module<br>
<br>
Please use trailing dot and "Mail:" prefix for mail-related<br>
changes.<br>
<br>
That is, I would like to see something like this here:<br>
<br>
Mail: missing ngx_ssl_ecdh_curve() call.<br>
<br>
><br>
> diff -r 7ffe7f3be404 -r f6c3e219648f src/mail/ngx_mail_ssl_module.c<br>
> --- a/src/mail/ngx_mail_ssl_module.c  Thu May 09 10:54:28 2013 +0200<br>
> +++ b/src/mail/ngx_mail_ssl_module.c  Fri May 10 16:53:45 2013 +0200<br>
> @@ -308,6 +308,10 @@<br>
>          return NGX_CONF_ERROR;<br>
>      }<br>
><br>
> +    if (ngx_ssl_ecdh_curve(cf, &conf->ssl, &conf->ecdh_curve) != NGX_OK) {<br>
> +        return NGX_CONF_ERROR;<br>
> +    }<br>
> +<br>
>      ngx_conf_merge_value(conf->builtin_session_cache,<br>
>                           prev->builtin_session_cache, NGX_SSL_NONE_SCACHE);<br>
><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Maxim Dounin<br>
<a href="http://nginx.org/en/donation.html" target="_blank">http://nginx.org/en/donation.html</a><br>
</font></span></blockquote></div><br></div>