Received: (from majordom@localhost)
	by hyperreal.org (8.8.5/8.8.5) id PAA16752;
	Fri, 15 Aug 1997 15:05:47 -0700 (PDT)
Received: from twinlark.arctic.org (twinlark.arctic.org [204.62.130.91])
	by hyperreal.org (8.8.5/8.8.5) with SMTP id PAA16725
	for <new-httpd@apache.org>; Fri, 15 Aug 1997 15:05:42 -0700 (PDT)
Received: (qmail 19152 invoked by uid 500); 15 Aug 1997 21:54:33 -0000
Date: Fri, 15 Aug 1997 14:54:33 -0700 (PDT)
From: Dean Gaudet <dgaudet@arctic.org>
To: new-httpd@apache.org
Subject: Re: [PATCH] Disabling logging based on envariable 
In-Reply-To: <199708152054.PAA16155@sierra.zyzzyva.com>
Message-ID: <Pine.LNX.3.95dg3.970815145200.18012I-100000@twinlark.arctic.org>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: new-httpd-owner@apache.org
Precedence: bulk
Reply-To: new-httpd@apache.org

Rob's example, and Ken's both add directives.  As usual I disagree with
that :)  My suggestion:

CustomLog logs/kaboom "%h - %u %t \"%r\" %s %b" !that_bloody_robot_again

That is, CustomLog takes a log filename, a log format string, and an
optional environment variable that must be present (or not present).  This
isn't a lot more work.

Dean

On Fri, 15 Aug 1997, Randy Terbush wrote:

> 
> > >Assuming this patch doesn't disable logging to specific logs, how about
> > >something like this..
> > >
> > ><Logfile>
> > >	LogFormat "%h - %u %t \"%r\" %s %b"
> > >	LogUnless env=that_bloody_robot_again
> > ></Logfile>
> > >
> > ><Logfile>
> > >	CustomLog logs/kaboom "%h - %u %t \"%r\" %s %b \"\" \"%{User-agent}i\""
> > >	LogIf status>=500
> > ></Logfile>
> > >
> > ><Logfile>
> > >	CustomLog logs/referer "%{referer}i -> %U"
> > >	LogIf env=referer_exists
> > >	LogUnless env=referer_is_not_local
> > ></Logfile>
> > 
> >     Very nice - but a *lot* more work.  Can we reserve this for a future
> >     (2.0?) iteration?
> > 
> >     #ken    :-)}
> 
> Not a -1, BUT...
> 
> Anything we put out there, we are somewhat obligated to maintain 
> some backward compatibility. Would it be worth putting off to be 
> able to do it as RobH suggests. Very clean example Rob.
> 
> 
> 
>