Re: svn commit: r1467433 - /httpd/httpd/branches/2.2.x/STATUS

[Daniel Ruggeri <DR...@primary.net>]

On 4/12/2013 2:32 PM, wrowe@apache.org wrote:

> Author: wrowe
> Date: Fri Apr 12 19:32:55 2013
> New Revision: 1467433
>
> URL: http://svn.apache.org/r1467433
> Log:
> Belated explantion for druggeri
>
> Modified:
>     httpd/httpd/branches/2.2.x/STATUS
>
> Modified: httpd/httpd/branches/2.2.x/STATUS
> URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=1467433&r1=1467432&r2=1467433&view=diff
> ==============================================================================
> --- httpd/httpd/branches/2.2.x/STATUS (original)
> +++ httpd/httpd/branches/2.2.x/STATUS Fri Apr 12 19:32:55 2013
> @@ -189,7 +189,8 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
>       2.2.x patch: http://people.apache.org/~druggeri/patches/AuthLDAPBindPasswordExec-2.2.patch
>                    (20130119 - updated to include minor mmn bump)
>       +1: druggeri
> -     -1: wrowe (switch to +1 once ap_get_exec_line is made thread-safe)
> +     -1: wrowe (switch to +1 once ap_get_exec_line is made thread-safe, right
> +               now it uses a static which is verboten in reentrant code.)
>                 kudos for using apr_tokenize_to_argv to allow spaces in args.
>  
>    * mod_ssl: Quiet FIPS mode weak keys disabled and FIPS not selected emits

Argh... That makes sense now. Missed that as I ripped the code out of mod_ssl. I have updated the patches at:
http://people.apache.org/~druggeri/patches/AuthLDAPBindPasswordExec-2.2.patch
http://people.apache.org/~druggeri/patches/AuthLDAPBindPasswordExec-2.4.patch

... and corrected trunk + notes in STATUS.

Thanks, Bill. If that works for you, I've also proposed this in 2.4 and could use votes there, too :)

--
Daniel Ruggeri