From: Jeff Trawick Date: Thu, 27 May 2004 23:35:41 +0000 (+0000) Subject: Pass environment variables through to piped loggers, resolving X-Git-Tag: pre_ajp_proxy~211 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0619fe865902d6bd66ab7133e81d124df2bf650e;p=apache Pass environment variables through to piped loggers, resolving a regression since 1.3. Submitted by: Ken Coar, Jeff Trawick git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@103796 13f79535-47bb-0310-9956-ffa450edef68 --- diff --git a/CHANGES b/CHANGES index 0d430eb170..1e62eb5090 100644 --- a/CHANGES +++ b/CHANGES @@ -2,6 +2,9 @@ Changes with Apache 2.1.0-dev [Remove entries to the current 2.0 section below, when backported] + *) Pass environment variables through to piped loggers, resolving + a regression since 1.3. [Ken Coar, Jeff Trawick] + *) Enable the option to support anonymous shared memory in mod_ldap. This makes the cache work on Linux again. [Graham Leggett] diff --git a/server/log.c b/server/log.c index fcef37aa64..301b8fcdcc 100644 --- a/server/log.c +++ b/server/log.c @@ -197,6 +197,8 @@ static int log_child(apr_pool_t *p, const char *progname, apr_proc_t *procnew; if (((rc = apr_procattr_create(&procattr, p)) == APR_SUCCESS) + && ((rc == apr_procattr_cmdtype_set(procattr, + APR_PROGRAM_ENV)) == APR_SUCCESS) && ((rc = apr_procattr_io_set(procattr, APR_FULL_BLOCK, APR_NO_PIPE, @@ -722,6 +724,8 @@ static int piped_log_spawn(piped_log *pl) apr_status_t status; if (((status = apr_procattr_create(&procattr, pl->p)) != APR_SUCCESS) || + ((status = apr_procattr_cmdtype_set(procattr, + APR_PROGRAM_ENV)) != APR_SUCCESS) || ((status = apr_procattr_child_in_set(procattr, ap_piped_log_read_fd(pl), ap_piped_log_write_fd(pl)))