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

Michael Kliewe info at phpgangsta.de
Mon Mar 2 12:12:44 UTC 2015


Hi Maxim,

with your changes there is a problem:
nginx now just sends the header if the connection is encrypted. If the connection is not encrypted, then there is no header sent to the auth script.
In the auth script I cannot distinguish between "user did not use encryption" and "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 the unpatched nginx version again instead of self-compiling it every time ;-)
> 
> Michael
> 
> Am 25.02.2015 um 16:28 schrieb Maxim Dounin:
>> Hello!
>> 
>> On Thu, Feb 05, 2015 at 04:00:28PM +0300, Maxim Dounin wrote:
>> 
>>> Hello!
>>> 
>>> On Wed, Feb 04, 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 values as you like. It would be very nice
>>>> if you could please merge it into nginx.
>>> I'm planning to look into this patch and other mail SSL
>>> improvements once I've done with unbuffered upload feature I'm
>>> currently working on.
>> Just an update: a patch to address this was committed, see
>> http://hg.nginx.org/nginx/rev/3b3f789655dc.
>> 
>> Thanks Filipe for the original patch, and thanks Michael for
>> prodding this.
>> 
> 
> _______________________________________________
> nginx-devel mailing list
> nginx-devel at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel



More information about the nginx-devel mailing list