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

Mon Mar 2 13:12:48 UTC 2015

2015-03-02 13:12 GMT+01:00 Michael Kliewe <info at phpgangsta.de>:
> 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
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel