httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rainer Jung <rainer.j...@kippdata.de>
Subject Re: [VOTE] release httpd mod_fcgid-2.3.1?
Date Wed, 09 Sep 2009 21:27:19 GMT
On 09.09.2009 01:48, William A. Rowe, Jr. wrote:
> mod_fcgid freinds;
> 
> Please fetch up the newly minted mod_fcgid-2.3.1.tar.gz (or bz2)
> or the win32 suitable package mod_fcgid-2.3.1-crlf.zip from:
> 
>   http://httpd.apache.org/dev/dist/mod_fcgid/
> 
> review, take it for a spin, and cast your choice
> 
>   [ ] -1 for any release of 2.3.1 (regressed from 2.3.0?)
>   [X] +1 to release as 2.3.1-beta
>   [ ] +1 to release as 2.3.1-GA

> Note no configuration updates occur yet on make install.  Not sure what our
> 'example' aught to look like, yet.  Note no documentation was donated, so
> there is a reference to all of the directives taken from the cmd_rec array,
> but this needs attention from fans to fill in the text and correct a few
> more typos.

Tested against httpd 2.2.x on Solaris 8.

Signing good, hash files are in a format at least my md5sum and sha1sum
do not understand how to check.

configure runs fine.
Some minor warnings during make (all Solaris-specific):

fcgid_proctbl_unix.c:261: warning: format '%d' expects type 'int', but
argument 7 has type 'pid_t'
fcgid_proctbl_unix.c:268: warning: format '%d' expects type 'int', but
argument 7 has type 'pid_t'
fcgid_proctbl_unix.c:280: warning: format '%d' expects type 'int', but
argument 7 has type 'pid_t'
fcgid_pm_unix.c:239: warning: format '%d' expects type 'int', but
argument 7 has type 'pid_t'
fcgid_pm_unix.c:268: warning: format '%d' expects type 'int', but
argument 7 has type 'pid_t'
fcgid_proc_unix.c:229: warning: format '%d' expects type 'int', but
argument 5 has type 'pid_t'
fcgid_proc_unix.c:865: warning: format '%d' expects type 'int', but
argument 8 has type 'pid_t'

Tested with a simple echo server using ab and a few thousand requests.
Looks reliable.

I think it's still beta, because the docs are not enough to get one
started. Especially the description for FCGIWrapper is strange:

- I wasn't able to use it without the optional (?) second param:

In file fcgid_conf.c:

762 const char *set_wrapper_config(cmd_parms * cmd, void *dirconfig,
763                                const char *wrapperpath,
764                                const char *extension)
765 {
...
775     /* Sanity check */
776     if (wrapperpath == NULL || extension == NULL
-----------------------------------^^^^^^^^^^^^^^^^^
777         || *extension != '.' || *(extension + 1) == '\0'
778     ...
779         return "Invalid wrapper file extension";

although it is declared as

    AP_INIT_TAKE12("FCGIWrapper", set_wrapper_config, NULL,
                   RSRC_CONF | ACCESS_CONF | OR_FILEINFO,
                   "The CGI wrapper setting"),

- the needed "extension" is completely undocumented

- the path of wrapperpath is not resolved relative to the server root

Regards,

Rainer

Mime
View raw message