You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Christophe Jaillet <ch...@wanadoo.fr> on 2018/06/29 19:31:39 UTC

Re: svn commit: r1834671 - in /httpd/httpd/branches/2.4.x: CHANGES docs/manual/mod/mod_md.xml modules/md/md_crypt.c modules/md/md_json.c modules/md/md_version.h modules/md/mod_md.c modules/md/mod_md_config.c

Le 29/06/2018 à 13:53, icing@apache.org a écrit :
> Author: icing
> Date: Fri Jun 29 11:53:50 2018
> New Revision: 1834671
> 
> URL: http://svn.apache.org/viewvc?rev=1834671&view=rev
> Log:
> On the 2.4.x branch:
> 
> backport of current mod_md version and documentation.
> 
> 
> Modified:
>      httpd/httpd/branches/2.4.x/CHANGES
>      httpd/httpd/branches/2.4.x/docs/manual/mod/mod_md.xml
>      httpd/httpd/branches/2.4.x/modules/md/md_crypt.c
>      httpd/httpd/branches/2.4.x/modules/md/md_json.c
>      httpd/httpd/branches/2.4.x/modules/md/md_version.h
>      httpd/httpd/branches/2.4.x/modules/md/mod_md.c
>      httpd/httpd/branches/2.4.x/modules/md/mod_md_config.c
> 

Hi,

according to 2.4.x/STATUS, mod_md is not CTR.
Maybe, STATUS should be updated to add an exception for mod_md? (or 
maybe even for any module marked as experimental?)
Otherwise, this should go through STATUS first.

CJ

Re: svn commit: r1834671 - in /httpd/httpd/branches/2.4.x: CHANGES docs/manual/mod/mod_md.xml modules/md/md_crypt.c modules/md/md_json.c modules/md/md_version.h modules/md/mod_md.c modules/md/mod_md_config.c

Posted by Stefan Eissing <st...@greenbytes.de>.

> Am 03.07.2018 um 13:54 schrieb Yann Ylavic <yl...@gmail.com>:
> 
> On Tue, Jul 3, 2018 at 12:53 PM, Jim Jagielski <ji...@jagunet.com> wrote:
>> 
>> 
>>> On Jul 2, 2018, at 1:40 PM, Christophe Jaillet <ch...@wanadoo.fr> wrote:
>>> 
>>> Le 02/07/2018 à 17:36, William A Rowe Jr a écrit :
>>>> On Mon, Jul 2, 2018 at 8:25 AM, Stefan Eissing <stefan.eissing@greenbytes.de <ma...@greenbytes.de>> wrote:
>>>>   I thought experimental == CTR, but if this is separate then I‘ll go
>>>>   through the votes. Just let me know what you prefer. I basically thought the same thing, but it is clearly spelled out in STATUS.
>>>> We aught to adjust this to reflect the eventual consensus;
>>>>  * Current exceptions for RTC for this branch:
>>>>    . mod_proxy_http2
>>>>    . mod_lua
>>>>    . documentation
>>>>    . non-Unix build
>>>>    . non-Unix, single-platform code
>>> 
>>> I don't have a strong opinion about it, that's why I proposed to update STATUS to add an exception for mod_md? (or maybe even for any module marked as experimental?)
>>> 
>> 
>> +1 (for CTR for both mod_md *as well as* any module marked experimental)
> 
> +1

+1

Re: svn commit: r1834671 - in /httpd/httpd/branches/2.4.x: CHANGES docs/manual/mod/mod_md.xml modules/md/md_crypt.c modules/md/md_json.c modules/md/md_version.h modules/md/mod_md.c modules/md/mod_md_config.c

Posted by Yann Ylavic <yl...@gmail.com>.
On Tue, Jul 3, 2018 at 12:53 PM, Jim Jagielski <ji...@jagunet.com> wrote:
>
>
>> On Jul 2, 2018, at 1:40 PM, Christophe Jaillet <ch...@wanadoo.fr> wrote:
>>
>> Le 02/07/2018 à 17:36, William A Rowe Jr a écrit :
>>> On Mon, Jul 2, 2018 at 8:25 AM, Stefan Eissing <stefan.eissing@greenbytes.de <ma...@greenbytes.de>> wrote:
>>>    I thought experimental == CTR, but if this is separate then I‘ll go
>>>    through the votes. Just let me know what you prefer. I basically thought the same thing, but it is clearly spelled out in STATUS.
>>> We aught to adjust this to reflect the eventual consensus;
>>>   * Current exceptions for RTC for this branch:
>>>     . mod_proxy_http2
>>>     . mod_lua
>>>     . documentation
>>>     . non-Unix build
>>>     . non-Unix, single-platform code
>>
>> I don't have a strong opinion about it, that's why I proposed to update STATUS to add an exception for mod_md? (or maybe even for any module marked as experimental?)
>>
>
> +1 (for CTR for both mod_md *as well as* any module marked experimental)

+1

Re: svn commit: r1834671 - in /httpd/httpd/branches/2.4.x: CHANGES docs/manual/mod/mod_md.xml modules/md/md_crypt.c modules/md/md_json.c modules/md/md_version.h modules/md/mod_md.c modules/md/mod_md_config.c

Posted by Jim Jagielski <ji...@jaguNET.com>.

> On Jul 2, 2018, at 1:40 PM, Christophe Jaillet <ch...@wanadoo.fr> wrote:
> 
> Le 02/07/2018 à 17:36, William A Rowe Jr a écrit :
>> On Mon, Jul 2, 2018 at 8:25 AM, Stefan Eissing <stefan.eissing@greenbytes.de <ma...@greenbytes.de>> wrote:
>>    I thought experimental == CTR, but if this is separate then I‘ll go
>>    through the votes. Just let me know what you prefer. I basically thought the same thing, but it is clearly spelled out in STATUS.
>> We aught to adjust this to reflect the eventual consensus;
>>   * Current exceptions for RTC for this branch:
>>     . mod_proxy_http2
>>     . mod_lua
>>     . documentation
>>     . non-Unix build
>>     . non-Unix, single-platform code
> 
> I don't have a strong opinion about it, that's why I proposed to update STATUS to add an exception for mod_md? (or maybe even for any module marked as experimental?)
> 

+1 (for CTR for both mod_md *as well as* any module marked experimental)


Re: svn commit: r1834671 - in /httpd/httpd/branches/2.4.x: CHANGES docs/manual/mod/mod_md.xml modules/md/md_crypt.c modules/md/md_json.c modules/md/md_version.h modules/md/mod_md.c modules/md/mod_md_config.c

Posted by Christophe Jaillet <ch...@wanadoo.fr>.
Le 02/07/2018 à 17:36, William A Rowe Jr a écrit :
> On Mon, Jul 2, 2018 at 8:25 AM, Stefan Eissing 
> <stefan.eissing@greenbytes.de <ma...@greenbytes.de>> wrote:
> 
>     I thought experimental == CTR, but if this is separate then I‘ll go
>     through the votes. Just let me know what you prefer. 
> 
> 
> I basically thought the same thing, but it is clearly spelled out in STATUS.
> We aught to adjust this to reflect the eventual consensus;
> 
>    * Current exceptions for RTC for this branch:
>      . mod_proxy_http2
>      . mod_lua
>      . documentation
>      . non-Unix build
>      . non-Unix, single-platform code

I don't have a strong opinion about it, that's why I proposed to update 
STATUS to add an exception for mod_md? (or maybe even for any module 
marked as experimental?)

A grep in 2.4.x, shows the following modules marked as Experimental:
mod_allowmethods.xml:<status>Experimental</status>
mod_dialup.xml:<status>Experimental</status>
mod_echo.xml:<status>Experimental</status>
mod_example_hooks.xml:<status>Experimental</status>
mod_file_cache.xml:<status>Experimental</status>
mod_heartbeat.xml:<status>Experimental</status>
mod_heartmonitor.xml:<status>Experimental</status>
mod_lbmethod_heartbeat.xml:<status>Experimental</status>
mod_log_debug.xml:<status>Experimental</status>
mod_lua.xml:<status>Experimental</status>
mod_privileges.xml:<status>Experimental</status>
mod_sed.xml:<status>Experimental</status>
mod_session_crypto.xml:<status>Experimental</status>

I suspect, that some should not be marked as Experimental anymore.


mod_md and mod_proxy_http2 don't have an Experimental status, only a 
red-lined warning at the beginning of its doc.



Maybe something should be added in the module list to easily spot the 
one with an Experimental status and a note explaining what this mean. 
(i.e., if I'm correct, the code is considered as stable, but its 
implementation and/or API and/or directives are subject to change in 
future release)

CJ

Re: svn commit: r1834671 - in /httpd/httpd/branches/2.4.x: CHANGES docs/manual/mod/mod_md.xml modules/md/md_crypt.c modules/md/md_json.c modules/md/md_version.h modules/md/mod_md.c modules/md/mod_md_config.c

Posted by William A Rowe Jr <wr...@rowe-clan.net>.
On Mon, Jul 2, 2018 at 8:25 AM, Stefan Eissing <stefan.eissing@greenbytes.de
> wrote:

> I thought experimental == CTR, but if this is separate then I‘ll go
> through the votes. Just let me know what you prefer.


I basically thought the same thing, but it is clearly spelled out in STATUS.
We aught to adjust this to reflect the eventual consensus;

  * Current exceptions for RTC for this branch:
    . mod_proxy_http2
    . mod_lua
    . documentation
    . non-Unix build
    . non-Unix, single-platform code

Re: svn commit: r1834671 - in /httpd/httpd/branches/2.4.x: CHANGES docs/manual/mod/mod_md.xml modules/md/md_crypt.c modules/md/md_json.c modules/md/md_version.h modules/md/mod_md.c modules/md/mod_md_config.c

Posted by Stefan Eissing <st...@greenbytes.de>.
I thought experimental == CTR, but if this is separate then I‘ll go through the votes. Just let me know what you prefer. 

> Am 29.06.2018 um 21:31 schrieb Christophe Jaillet <ch...@wanadoo.fr>:
> 
>> Le 29/06/2018 à 13:53, icing@apache.org a écrit :
>> Author: icing
>> Date: Fri Jun 29 11:53:50 2018
>> New Revision: 1834671
>> URL: http://svn.apache.org/viewvc?rev=1834671&view=rev
>> Log:
>> On the 2.4.x branch:
>> backport of current mod_md version and documentation.
>> Modified:
>>     httpd/httpd/branches/2.4.x/CHANGES
>>     httpd/httpd/branches/2.4.x/docs/manual/mod/mod_md.xml
>>     httpd/httpd/branches/2.4.x/modules/md/md_crypt.c
>>     httpd/httpd/branches/2.4.x/modules/md/md_json.c
>>     httpd/httpd/branches/2.4.x/modules/md/md_version.h
>>     httpd/httpd/branches/2.4.x/modules/md/mod_md.c
>>     httpd/httpd/branches/2.4.x/modules/md/mod_md_config.c
> 
> Hi,
> 
> according to 2.4.x/STATUS, mod_md is not CTR.
> Maybe, STATUS should be updated to add an exception for mod_md? (or maybe even for any module marked as experimental?)
> Otherwise, this should go through STATUS first.
> 
> CJ